From 228fe087839889417a38d2e6f94a8c9e85cba054 Mon Sep 17 00:00:00 2001 From: austinried <4966622+austinried@users.noreply.github.com> Date: Wed, 23 Sep 2026 07:11:14 +0000 Subject: [PATCH] trim the library header, enlarge artist art, fix album title color --- .../com/subtracks/ui/library/ArtistDetailScreen.kt | 11 ++--------- .../kotlin/com/subtracks/ui/library/LibraryScreen.kt | 4 ++-- 2 files changed, 4 insertions(+), 11 deletions(-) diff --git a/app/src/main/kotlin/com/subtracks/ui/library/ArtistDetailScreen.kt b/app/src/main/kotlin/com/subtracks/ui/library/ArtistDetailScreen.kt index 62483114..a0e271f1 100644 --- a/app/src/main/kotlin/com/subtracks/ui/library/ArtistDetailScreen.kt +++ b/app/src/main/kotlin/com/subtracks/ui/library/ArtistDetailScreen.kt @@ -42,7 +42,7 @@ import com.subtracks.ui.components.CoverArt import org.koin.compose.viewmodel.koinViewModel import org.koin.core.parameter.parametersOf -private val ART_HEIGHT = 320.dp +private val ART_HEIGHT = 420.dp private val SCRIM_HEIGHT = 128.dp @Composable @@ -102,14 +102,6 @@ fun ArtistDetailScreen( ) } } - item(span = { GridItemSpan(maxLineSpan) }) { - Text( - text = "${albums.size} albums", - style = MaterialTheme.typography.bodyMedium, - color = MaterialTheme.colorScheme.onSurfaceVariant, - modifier = Modifier.padding(start = 16.dp, end = 16.dp, bottom = 4.dp), - ) - } itemsIndexed(albums, key = { _, album -> album.id }) { index, album -> Column( modifier = @@ -127,6 +119,7 @@ fun ArtistDetailScreen( Text( text = album.name, style = MaterialTheme.typography.bodyMedium, + color = MaterialTheme.colorScheme.onBackground, maxLines = 1, overflow = TextOverflow.Ellipsis, modifier = Modifier.padding(top = 6.dp), diff --git a/app/src/main/kotlin/com/subtracks/ui/library/LibraryScreen.kt b/app/src/main/kotlin/com/subtracks/ui/library/LibraryScreen.kt index 3f744eaa..90c23629 100644 --- a/app/src/main/kotlin/com/subtracks/ui/library/LibraryScreen.kt +++ b/app/src/main/kotlin/com/subtracks/ui/library/LibraryScreen.kt @@ -155,13 +155,13 @@ fun LibraryScreen( } val density = LocalDensity.current - val titleHeight = TopAppBarDefaults.TopAppBarExpandedHeight val titleLineHeight = with(density) { MaterialTheme.typography.headlineLarge.lineHeight .toDp() } - val titleTop = ((titleHeight - titleLineHeight) / 2).coerceAtLeast(0.dp) + val titleTop = ((TopAppBarDefaults.TopAppBarExpandedHeight - titleLineHeight) / 2).coerceAtLeast(0.dp) + val titleHeight = titleTop + titleLineHeight + 4.dp val scrollBehavior = TopAppBarDefaults.enterAlwaysScrollBehavior() SideEffect { scrollBehavior.state.heightOffsetLimit = -with(density) { titleHeight.toPx() } } val titleFraction = scrollBehavior.state.collapsedFraction