From 2e2a01e25c3f0b1581efd1bc84611a300d4e950f Mon Sep 17 00:00:00 2001 From: Ed George Date: Mon, 28 Mar 2022 19:17:36 +0100 Subject: [PATCH] Update BottomBar height within Article Screen (#764) * Update BottomBar height within Article Screen Fixes #763 * [JetNews] Swap the order of modifiers Swap the order of the windowInsetsPadding and height modifiers to fix incorrect layout * Fix spotless error --- .../main/java/com/example/jetnews/ui/article/ArticleScreen.kt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/JetNews/app/src/main/java/com/example/jetnews/ui/article/ArticleScreen.kt b/JetNews/app/src/main/java/com/example/jetnews/ui/article/ArticleScreen.kt index 9a65fb0706..365f5c2fe2 100644 --- a/JetNews/app/src/main/java/com/example/jetnews/ui/article/ArticleScreen.kt +++ b/JetNews/app/src/main/java/com/example/jetnews/ui/article/ArticleScreen.kt @@ -213,9 +213,9 @@ private fun BottomBar( Row( verticalAlignment = Alignment.CenterVertically, modifier = Modifier + .windowInsetsPadding(WindowInsets.navigationBars.only(WindowInsetsSides.Vertical)) .height(56.dp) .fillMaxWidth() - .windowInsetsPadding(WindowInsets.navigationBars.only(WindowInsetsSides.Vertical)) ) { FavoriteButton(onClick = onUnimplementedAction) BookmarkButton(isBookmarked = isFavorite, onClick = onToggleFavorite)