From 1106714707a02dad8ad69b11b2f3f76591b7fe7f Mon Sep 17 00:00:00 2001
From: Simon Marquis <contact@simon-marquis.fr>
Date: Fri, 13 May 2022 00:36:15 +0200
Subject: [PATCH] Move `AuthorsCarousel`'s padding into `AuthorItem`

Fixes #18
---
 .../apps/nowinandroid/feature/foryou/AuthorsCarousel.kt  | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/feature-foryou/src/main/java/com/google/samples/apps/nowinandroid/feature/foryou/AuthorsCarousel.kt b/feature-foryou/src/main/java/com/google/samples/apps/nowinandroid/feature/foryou/AuthorsCarousel.kt
index df6b8bbda..7aabeee37 100644
--- a/feature-foryou/src/main/java/com/google/samples/apps/nowinandroid/feature/foryou/AuthorsCarousel.kt
+++ b/feature-foryou/src/main/java/com/google/samples/apps/nowinandroid/feature/foryou/AuthorsCarousel.kt
@@ -63,7 +63,6 @@ fun AuthorsCarousel(
                 onAuthorClick = { following ->
                     onAuthorClick(followableAuthor.author.id, following)
                 },
-                modifier = Modifier.padding(8.dp)
             )
         }
     }
@@ -90,12 +89,16 @@ fun AuthorItem(
                 role = Role.Button,
                 onValueChange = { newFollowing -> onAuthorClick(newFollowing) },
             )
-            .sizeIn(maxWidth = 48.dp)
+            .padding(vertical = 8.dp, horizontal = 4.dp)
+            .sizeIn(maxWidth = 56.dp)
             .semantics(mergeDescendants = true) {
                 stateDescription = "$followDescription ${author.name}"
             }
     ) {
-        Box(modifier = Modifier.fillMaxWidth()) {
+        Box(
+            modifier = Modifier.fillMaxWidth(),
+            contentAlignment = Alignment.Center,
+        ) {
             AsyncImage(
                 modifier = Modifier
                     .size(48.dp)