Merge pull request #1673 from android/dt/fix-icons

dturner · web-flow · commit ecea15d3e406 · 2024-10-21T16:45:20.000+01:00
Reduce icon sizes
diff --git a/core/ui/src/main/kotlin/com/google/samples/apps/nowinandroid/core/ui/InterestsItem.kt b/core/ui/src/main/kotlin/com/google/samples/apps/nowinandroid/core/ui/InterestsItem.kt
@@ -54,7 +54,7 @@ fun InterestsItem(
 ) {
     ListItem(
         leadingContent = {
-            InterestsIcon(topicImageUrl, iconModifier.size(64.dp))
+            InterestsIcon(topicImageUrl, iconModifier.size(48.dp))
         },
         headlineContent = {
             Text(text = name)
diff --git a/feature/topic/src/main/kotlin/com/google/samples/apps/nowinandroid/feature/topic/TopicScreen.kt b/feature/topic/src/main/kotlin/com/google/samples/apps/nowinandroid/feature/topic/TopicScreen.kt
@@ -212,7 +212,7 @@ private fun TopicHeader(name: String, description: String, imageUrl: String) {
             contentDescription = null,
             modifier = Modifier
                 .align(Alignment.CenterHorizontally)
-                .size(216.dp)
+                .size(132.dp)
                 .padding(bottom = 12.dp),
         )
         Text(name, style = MaterialTheme.typography.displayMedium)

Original file line number	Diff line number	Diff line change
`@@ -212,7 +212,7 @@ private fun TopicHeader(name: String, description: String, imageUrl: String) {`
`212`	`212`	`contentDescription = null,`
`213`	`213`	`modifier = Modifier`
`214`	`214`	`.align(Alignment.CenterHorizontally)`
`215`		`- .size(216.dp)`
	`215`	`+ .size(132.dp)`
`216`	`216`	`.padding(bottom = 12.dp),`
`217`	`217`	`)`
`218`	`218`	`Text(name, style = MaterialTheme.typography.displayMedium)`