From 3668d8f7d1ccfc85b1e82b253adabe0d9f4ee1cd Mon Sep 17 00:00:00 2001 From: Simon Marquis Date: Thu, 19 May 2022 21:34:19 +0200 Subject: [PATCH] Restore the expected layout dimensions --- .../nowinandroid/feature/foryou/AuthorsCarousel.kt | 10 ++++++---- 1 file changed, 6 insertions(+), 4 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 7aabeee37..a8247cd4f 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 @@ -21,7 +21,6 @@ import androidx.compose.foundation.layout.Column import androidx.compose.foundation.layout.Spacer import androidx.compose.foundation.layout.fillMaxWidth import androidx.compose.foundation.layout.height -import androidx.compose.foundation.layout.padding import androidx.compose.foundation.layout.size import androidx.compose.foundation.layout.sizeIn import androidx.compose.foundation.lazy.LazyRow @@ -55,7 +54,11 @@ fun AuthorsCarousel( onAuthorClick: (String, Boolean) -> Unit, modifier: Modifier = Modifier ) { - LazyRow(modifier) { + LazyRow( + modifier = modifier, + contentPadding = PaddingValues(horizontal = 24.dp), + horizontalArrangement = Arrangement.spacedBy(24.dp) + ) { items(items = authors, key = { item -> item.author.id }) { followableAuthor -> AuthorItem( author = followableAuthor.author, @@ -89,8 +92,7 @@ fun AuthorItem( role = Role.Button, onValueChange = { newFollowing -> onAuthorClick(newFollowing) }, ) - .padding(vertical = 8.dp, horizontal = 4.dp) - .sizeIn(maxWidth = 56.dp) + .sizeIn(maxWidth = 48.dp) .semantics(mergeDescendants = true) { stateDescription = "$followDescription ${author.name}" }