Skip to content

Commit 50bc538

Browse files
Fix #285 remove gray box behind avatar and fix settings header (#289)
Made the settings top app bar transparent so it matches the profile screen and doesn't show a weird gray bar when collapsed. Also added a proper background to the AvatarHeader box so the placeholder avatar blends in instead of showing a gray rectangle.
1 parent 8570978 commit 50bc538

2 files changed

Lines changed: 4 additions & 5 deletions

File tree

presentation/src/main/java/org/monogram/presentation/core/ui/AvatarHeader.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ fun AvatarHeader(
6464
fallbackPath = fallbackPath?.takeIf { it.isNotBlank() } ?: cachedFallbackPath
6565
)
6666

67-
Box(modifier = combinedModifier) {
67+
Box(modifier = combinedModifier.background(MaterialTheme.colorScheme.surfaceContainerHighest)) {
6868
val avatarFile = resolvedPath?.let { File(it) }
6969

7070
if (avatarFile != null && avatarFile.exists()) {

presentation/src/main/java/org/monogram/presentation/settings/settings/SettingsContent.kt

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -585,10 +585,9 @@ fun SettingsContent(component: SettingsComponent) {
585585
}
586586
},
587587
colors = TopAppBarDefaults.topAppBarColors(
588-
containerColor = dynamicContainerColorTopBar,
589-
scrolledContainerColor = Color.Transparent,
590-
591-
)
588+
containerColor = Color.Transparent,
589+
scrolledContainerColor = Color.Transparent
590+
)
592591
)
593592
}
594593
) { padding ->

0 commit comments

Comments
 (0)