From aefbf8a9c0ef60016abf70fe11e371c46a150325 Mon Sep 17 00:00:00 2001 From: YvesKalume Date: Fri, 13 May 2022 17:35:10 +0200 Subject: [PATCH 1/4] fix distorted buttons in nia-catalog --- app-nia-catalog/build.gradle.kts | 1 + .../samples/apps/niacatalog/ui/Catalog.kt | 33 ++++++++++--------- 2 files changed, 18 insertions(+), 16 deletions(-) diff --git a/app-nia-catalog/build.gradle.kts b/app-nia-catalog/build.gradle.kts index dc8d4ffc0..8ab94c29f 100644 --- a/app-nia-catalog/build.gradle.kts +++ b/app-nia-catalog/build.gradle.kts @@ -35,4 +35,5 @@ dependencies { implementation(project(":core-ui")) implementation(libs.androidx.activity.compose) + implementation(libs.accompanist.flowlayout) } \ No newline at end of file diff --git a/app-nia-catalog/src/main/java/com/google/samples/apps/niacatalog/ui/Catalog.kt b/app-nia-catalog/src/main/java/com/google/samples/apps/niacatalog/ui/Catalog.kt index c4f66166a..0513cb690 100644 --- a/app-nia-catalog/src/main/java/com/google/samples/apps/niacatalog/ui/Catalog.kt +++ b/app-nia-catalog/src/main/java/com/google/samples/apps/niacatalog/ui/Catalog.kt @@ -37,6 +37,7 @@ import androidx.compose.runtime.setValue import androidx.compose.ui.Modifier import androidx.compose.ui.res.painterResource import androidx.compose.ui.unit.dp +import com.google.accompanist.flowlayout.FlowRow import com.google.samples.apps.nowinandroid.core.ui.component.NiaDropdownMenuButton import com.google.samples.apps.nowinandroid.core.ui.component.NiaFilledButton import com.google.samples.apps.nowinandroid.core.ui.component.NiaFilterChip @@ -76,7 +77,7 @@ fun NiaCatalog() { } item { Text("Buttons", Modifier.padding(top = 16.dp)) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton(onClick = {}) { Text(text = "Enabled") } @@ -90,7 +91,7 @@ fun NiaCatalog() { } item { Text("Disabled buttons", Modifier.padding(top = 16.dp)) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, enabled = false @@ -113,7 +114,7 @@ fun NiaCatalog() { } item { Text("Buttons with leading icons", Modifier.padding(top = 16.dp)) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, text = { Text(text = "Enabled") }, @@ -139,7 +140,7 @@ fun NiaCatalog() { } item { Text("Disabled buttons with leading icons", Modifier.padding(top = 16.dp)) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, enabled = false, @@ -168,7 +169,7 @@ fun NiaCatalog() { } item { Text("Buttons with trailing icons", Modifier.padding(top = 16.dp)) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, text = { Text(text = "Enabled") }, @@ -194,7 +195,7 @@ fun NiaCatalog() { } item { Text("Disabled buttons with trailing icons", Modifier.padding(top = 16.dp)) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, enabled = false, @@ -223,7 +224,7 @@ fun NiaCatalog() { } item { Text("Small buttons", Modifier.padding(top = 16.dp)) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, small = true @@ -246,7 +247,7 @@ fun NiaCatalog() { } item { Text("Disabled small buttons", Modifier.padding(top = 16.dp)) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, enabled = false, @@ -272,7 +273,7 @@ fun NiaCatalog() { } item { Text("Small buttons with leading icons", Modifier.padding(top = 16.dp)) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, small = true, @@ -306,7 +307,7 @@ fun NiaCatalog() { ) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, enabled = false, @@ -338,7 +339,7 @@ fun NiaCatalog() { } item { Text("Small buttons with trailing icons", Modifier.padding(top = 16.dp)) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, small = true, @@ -372,7 +373,7 @@ fun NiaCatalog() { ) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, enabled = false, @@ -413,7 +414,7 @@ fun NiaCatalog() { } item { Text("Chips", Modifier.padding(top = 16.dp)) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { var firstChecked by remember { mutableStateOf(false) } NiaFilterChip( checked = firstChecked, @@ -437,7 +438,7 @@ fun NiaCatalog() { } item { Text("Toggle buttons", Modifier.padding(top = 16.dp)) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { var firstChecked by remember { mutableStateOf(false) } NiaToggleButton( checked = firstChecked, @@ -498,7 +499,7 @@ fun NiaCatalog() { } item { Text("View toggle", Modifier.padding(top = 16.dp)) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { var firstExpanded by remember { mutableStateOf(false) } NiaViewToggleButton( expanded = firstExpanded, @@ -517,7 +518,7 @@ fun NiaCatalog() { } item { Text("Tags", Modifier.padding(top = 16.dp)) } item { - Row(horizontalArrangement = Arrangement.spacedBy(16.dp)) { + FlowRow(mainAxisSpacing = 16.dp) { var firstFollowed by remember { mutableStateOf(false) } NiaTopicTag( followed = firstFollowed, From 72278b8cefbe18660c9aed7c210299c4e11c5b64 Mon Sep 17 00:00:00 2001 From: Sergio Belda Date: Tue, 17 May 2022 19:54:48 +0200 Subject: [PATCH 2/4] Add monochrome attribute to ic_launcher_round.xml in :app and :app-nia-catalog --- .../src/main/res/mipmap-anydpi-v26/ic_launcher_round.xml | 1 + app/src/main/res/mipmap-anydpi-v26/ic_launcher_round.xml | 1 + 2 files changed, 2 insertions(+) diff --git a/app-nia-catalog/src/main/res/mipmap-anydpi-v26/ic_launcher_round.xml b/app-nia-catalog/src/main/res/mipmap-anydpi-v26/ic_launcher_round.xml index ee103766b..28878a729 100644 --- a/app-nia-catalog/src/main/res/mipmap-anydpi-v26/ic_launcher_round.xml +++ b/app-nia-catalog/src/main/res/mipmap-anydpi-v26/ic_launcher_round.xml @@ -17,4 +17,5 @@ + \ No newline at end of file diff --git a/app/src/main/res/mipmap-anydpi-v26/ic_launcher_round.xml b/app/src/main/res/mipmap-anydpi-v26/ic_launcher_round.xml index ee103766b..28878a729 100644 --- a/app/src/main/res/mipmap-anydpi-v26/ic_launcher_round.xml +++ b/app/src/main/res/mipmap-anydpi-v26/ic_launcher_round.xml @@ -17,4 +17,5 @@ + \ No newline at end of file From b5d73bca5344b103285ce21328d657fa4744f59b Mon Sep 17 00:00:00 2001 From: YvesKalume Date: Wed, 18 May 2022 15:10:43 +0200 Subject: [PATCH 3/4] fix spotless format violations --- .../samples/apps/niacatalog/ui/Catalog.kt | 29 +++++++++---------- 1 file changed, 14 insertions(+), 15 deletions(-) diff --git a/app-nia-catalog/src/main/java/com/google/samples/apps/niacatalog/ui/Catalog.kt b/app-nia-catalog/src/main/java/com/google/samples/apps/niacatalog/ui/Catalog.kt index 0513cb690..30e73e17b 100644 --- a/app-nia-catalog/src/main/java/com/google/samples/apps/niacatalog/ui/Catalog.kt +++ b/app-nia-catalog/src/main/java/com/google/samples/apps/niacatalog/ui/Catalog.kt @@ -17,7 +17,6 @@ package com.google.samples.apps.niacatalog.ui import androidx.compose.foundation.layout.Arrangement -import androidx.compose.foundation.layout.Row import androidx.compose.foundation.layout.WindowInsets import androidx.compose.foundation.layout.add import androidx.compose.foundation.layout.asPaddingValues @@ -77,7 +76,7 @@ fun NiaCatalog() { } item { Text("Buttons", Modifier.padding(top = 16.dp)) } item { - FlowRow(mainAxisSpacing = 16.dp) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton(onClick = {}) { Text(text = "Enabled") } @@ -91,7 +90,7 @@ fun NiaCatalog() { } item { Text("Disabled buttons", Modifier.padding(top = 16.dp)) } item { - FlowRow(mainAxisSpacing = 16.dp) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, enabled = false @@ -114,7 +113,7 @@ fun NiaCatalog() { } item { Text("Buttons with leading icons", Modifier.padding(top = 16.dp)) } item { - FlowRow(mainAxisSpacing = 16.dp) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, text = { Text(text = "Enabled") }, @@ -140,7 +139,7 @@ fun NiaCatalog() { } item { Text("Disabled buttons with leading icons", Modifier.padding(top = 16.dp)) } item { - FlowRow(mainAxisSpacing = 16.dp) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, enabled = false, @@ -195,7 +194,7 @@ fun NiaCatalog() { } item { Text("Disabled buttons with trailing icons", Modifier.padding(top = 16.dp)) } item { - FlowRow(mainAxisSpacing = 16.dp) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, enabled = false, @@ -224,7 +223,7 @@ fun NiaCatalog() { } item { Text("Small buttons", Modifier.padding(top = 16.dp)) } item { - FlowRow(mainAxisSpacing = 16.dp) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, small = true @@ -247,7 +246,7 @@ fun NiaCatalog() { } item { Text("Disabled small buttons", Modifier.padding(top = 16.dp)) } item { - FlowRow(mainAxisSpacing = 16.dp) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, enabled = false, @@ -307,7 +306,7 @@ fun NiaCatalog() { ) } item { - FlowRow(mainAxisSpacing = 16.dp) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, enabled = false, @@ -339,7 +338,7 @@ fun NiaCatalog() { } item { Text("Small buttons with trailing icons", Modifier.padding(top = 16.dp)) } item { - FlowRow(mainAxisSpacing = 16.dp) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, small = true, @@ -373,7 +372,7 @@ fun NiaCatalog() { ) } item { - FlowRow(mainAxisSpacing = 16.dp) { + FlowRow(mainAxisSpacing = 16.dp) { NiaFilledButton( onClick = {}, enabled = false, @@ -414,7 +413,7 @@ fun NiaCatalog() { } item { Text("Chips", Modifier.padding(top = 16.dp)) } item { - FlowRow(mainAxisSpacing = 16.dp) { + FlowRow(mainAxisSpacing = 16.dp) { var firstChecked by remember { mutableStateOf(false) } NiaFilterChip( checked = firstChecked, @@ -438,7 +437,7 @@ fun NiaCatalog() { } item { Text("Toggle buttons", Modifier.padding(top = 16.dp)) } item { - FlowRow(mainAxisSpacing = 16.dp) { + FlowRow(mainAxisSpacing = 16.dp) { var firstChecked by remember { mutableStateOf(false) } NiaToggleButton( checked = firstChecked, @@ -499,7 +498,7 @@ fun NiaCatalog() { } item { Text("View toggle", Modifier.padding(top = 16.dp)) } item { - FlowRow(mainAxisSpacing = 16.dp) { + FlowRow(mainAxisSpacing = 16.dp) { var firstExpanded by remember { mutableStateOf(false) } NiaViewToggleButton( expanded = firstExpanded, @@ -518,7 +517,7 @@ fun NiaCatalog() { } item { Text("Tags", Modifier.padding(top = 16.dp)) } item { - FlowRow(mainAxisSpacing = 16.dp) { + FlowRow(mainAxisSpacing = 16.dp) { var firstFollowed by remember { mutableStateOf(false) } NiaTopicTag( followed = firstFollowed, From 9f10b5c4c6ed50d091178bdeffbc02366977f3ae Mon Sep 17 00:00:00 2001 From: Sergio Belda Date: Wed, 18 May 2022 16:49:04 +0200 Subject: [PATCH 4/4] Add cutout version of ic_launcher_foreground for drawable-v24 in app module in order to distinguish it from the nia-app-catalog version --- .../drawable-v24/ic_launcher_background.xml | 25 ++++++++++++++ .../drawable-v24/ic_launcher_foreground.xml | 34 +++++++++++++++++++ 2 files changed, 59 insertions(+) create mode 100644 app/src/main/res/drawable-v24/ic_launcher_background.xml create mode 100644 app/src/main/res/drawable-v24/ic_launcher_foreground.xml diff --git a/app/src/main/res/drawable-v24/ic_launcher_background.xml b/app/src/main/res/drawable-v24/ic_launcher_background.xml new file mode 100644 index 000000000..36fe340e6 --- /dev/null +++ b/app/src/main/res/drawable-v24/ic_launcher_background.xml @@ -0,0 +1,25 @@ + + + + + diff --git a/app/src/main/res/drawable-v24/ic_launcher_foreground.xml b/app/src/main/res/drawable-v24/ic_launcher_foreground.xml new file mode 100644 index 000000000..985a5c10b --- /dev/null +++ b/app/src/main/res/drawable-v24/ic_launcher_foreground.xml @@ -0,0 +1,34 @@ + + + + + + +