Makussu/subtracks
1
0
Fork 0
mirror of https://github.com/austinried/subtracks.git synced 2026-11-11 07:29:14 +01:00
Code Issues Packages Projects Releases Wiki Activity

ui: style artist section headers like album disc titles

Browse Source 
Use the same uppercase label-large semibold text with letter spacing as the
disc headers on the album screen.
This commit is contained in:
austinried
2026-10-19 04:49:33 +00:00
parent 5dd6e8dab9
commit 4f6a292e74
2 changed files with 11 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/src/main/kotlin/com/subtracks/ui/library/ArtistDetailScreen.kt
+6 -2
View File
@@ -72,10 +72,12 @@ import androidx.compose.ui.res.stringResource
import androidx.compose.ui.text.AnnotatedString
import androidx.compose.ui.text.TextMeasurer
import androidx.compose.ui.text.TextStyle
import androidx.compose.ui.text.font.FontWeight
import androidx.compose.ui.text.rememberTextMeasurer
import androidx.compose.ui.text.style.TextOverflow
import androidx.compose.ui.unit.Dp
import androidx.compose.ui.unit.dp
import androidx.compose.ui.unit.sp
import androidx.lifecycle.compose.collectAsStateWithLifecycle
import com.subtracks.R
import com.subtracks.data.model.Album
@@ -493,8 +495,10 @@ private fun ArtistSectionHeader(
verticalAlignment = Alignment.CenterVertically,
) {
Text(
text = text,
style = MaterialTheme.typography.titleMedium,
text = text.uppercase(),
style = MaterialTheme.typography.labelLarge,
fontWeight = FontWeight.SemiBold,
letterSpacing = 1.sp,
color = MaterialTheme.colorScheme.onSurfaceVariant,
modifier = Modifier.weight(1f),
)
app/src/test/kotlin/com/subtracks/ui/library/ArtistDetailScreenTest.kt
+5 -5
View File
@@ -70,7 +70,7 @@ class ArtistDetailScreenTest {
)
composeRule.onNodeWithText("Album artist · 1 album").assertExists()
composeRule.onNode(hasScrollAction()).performScrollToNode(hasText("Albums"))
composeRule.onNode(hasScrollAction()).performScrollToNode(hasText("ALBUMS"))
composeRule.onNodeWithContentDescription("More options").assertExists()
}
@@ -88,12 +88,12 @@ class ArtistDetailScreenTest {
composeRule.onNodeWithText("Album artist · 2 albums").assertExists()
composeRule.onNode(hasScrollAction()).performScrollToNode(hasText("Albums"))
composeRule.onNodeWithText("Albums").assertExists()
composeRule.onNode(hasScrollAction()).performScrollToNode(hasText("ALBUMS"))
composeRule.onNodeWithText("ALBUMS").assertExists()
composeRule.onNodeWithText("Kid A").assertExists()
composeRule.onNode(hasScrollAction()).performScrollToNode(hasText("Appears on"))
composeRule.onNodeWithText("Appears on").assertExists()
composeRule.onNode(hasScrollAction()).performScrollToNode(hasText("APPEARS ON"))
composeRule.onNodeWithText("APPEARS ON").assertExists()
composeRule.onNodeWithText("Guest Album").assertExists()
}