Set perpendicular item spacing for grids too

rubensousa · Sep 1, 2024 · 3366b4b · 3366b4b
1 parent 91c6e4a
commit 3366b4b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/...otlin/com/rubensousa/dpadrecyclerview/test/tests/spacing/DpadGridSpacingDecorationTest.kt b/...otlin/com/rubensousa/dpadrecyclerview/test/tests/spacing/DpadGridSpacingDecorationTest.kt
@@ -430,9 +430,9 @@ class DpadGridSpacingDecorationTest : DpadRecyclerViewTest() {
 
         // when
         onRecyclerView("Set spacing") { recyclerView ->
-            recyclerView.setItemSpacing(itemSpacing)
             recyclerView.setItemMinEdgeSpacing(minEdgeSpacing)
             recyclerView.setItemMaxEdgeSpacing(maxEdgeSpacing)
+            recyclerView.setItemSpacing(itemSpacing)
 
             // then
             assertThat(recyclerView.getSpacingDecoration()).isEqualTo(

diff --git a/dpadrecyclerview/src/main/java/com/rubensousa/dpadrecyclerview/DpadRecyclerView.kt b/dpadrecyclerview/src/main/java/com/rubensousa/dpadrecyclerview/DpadRecyclerView.kt
@@ -1006,7 +1006,10 @@ open class DpadRecyclerView @JvmOverloads constructor(
         updateSpacingDecoration { previousDecoration ->
             when {
                 previousDecoration is DpadGridSpacingDecoration -> {
-                    previousDecoration.copy(itemSpacing = spacing)
+                    previousDecoration.copy(
+                        itemSpacing = spacing,
+                        perpendicularItemSpacing = spacing
+                    )
                 }
 
                 previousDecoration is DpadLinearSpacingDecoration -> {