jenkins-bot has submitted this change and it was merged.

Change subject: Fix regression of clipping News items to padding.
......................................................................


Fix regression of clipping News items to padding.

One of our recent changes has brought back the issue of News items being
clipped to the padding/margin of their parent RecyclerView.

This correctly restores the no-clipping behavior, allowing the News items
to extend all the way to the edge of the card.

Change-Id: I42621c6cd075664bdca46e2b07e34d283dc6454f
---
M app/src/main/java/org/wikipedia/feed/view/HorizontalScrollingListCardView.java
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  jenkins-bot: Verified
  Niedzielski: Looks good to me, approved



diff --git 
a/app/src/main/java/org/wikipedia/feed/view/HorizontalScrollingListCardView.java
 
b/app/src/main/java/org/wikipedia/feed/view/HorizontalScrollingListCardView.java
index 8f53e45..12c554e 100644
--- 
a/app/src/main/java/org/wikipedia/feed/view/HorizontalScrollingListCardView.java
+++ 
b/app/src/main/java/org/wikipedia/feed/view/HorizontalScrollingListCardView.java
@@ -30,12 +30,12 @@
                 
R.dimen.view_horizontal_scrolling_list_card_item_margin_vertical));
         recyclerView.addOnItemTouchListener(new DontInterceptTouchListener());
         recyclerView.setNestedScrollingEnabled(false);
-        recyclerView.setClipChildren(false);
+        recyclerView.setClipToPadding(false);
         MarginLayoutParams params = (MarginLayoutParams) 
recyclerView.getLayoutParams();
         final int height = DimenUtil.roundedDpToPx(228);
         params.height = height;
-        final int margin = DimenUtil.roundedDpToPx(12);
-        params.setMargins(margin, 0, margin, 0);
+        final int padding = DimenUtil.roundedDpToPx(12);
+        recyclerView.setPadding(padding, 0, padding, 0);
     }
 
     protected abstract static class RecyclerAdapter<T>

-- 
To view, visit https://gerrit.wikimedia.org/r/312247
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I42621c6cd075664bdca46e2b07e34d283dc6454f
Gerrit-PatchSet: 2
Gerrit-Project: apps/android/wikipedia
Gerrit-Branch: master
Gerrit-Owner: Dbrant <dbr...@wikimedia.org>
Gerrit-Reviewer: BearND <bsitzm...@wikimedia.org>
Gerrit-Reviewer: Brion VIBBER <br...@wikimedia.org>
Gerrit-Reviewer: Mholloway <mhollo...@wikimedia.org>
Gerrit-Reviewer: Niedzielski <sniedziel...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to