From 5583c9c1d6d2ce061d77993baa251cbd52963264 Mon Sep 17 00:00:00 2001 From: austinried <4966622+austinried@users.noreply.github.com> Date: Tue, 20 Oct 2026 13:16:24 +0000 Subject: [PATCH] ui: centre the artist actions with the shuffle button Nudge the star and overflow row 8dp right, and restore the vertical offset, so the action row and shuffle button share a centre line. Keep the extra collapsed-bio spacing and the gap before the Albums header. --- .../com/subtracks/ui/library/ArtistDetailScreen.kt | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 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 13c84cd4..b1358339 100644 --- a/app/src/main/kotlin/com/subtracks/ui/library/ArtistDetailScreen.kt +++ b/app/src/main/kotlin/com/subtracks/ui/library/ArtistDetailScreen.kt @@ -147,9 +147,9 @@ private val SHUFFLE_RESERVE = 86.dp private val SHUFFLE_HANG = SHUFFLE_HEIGHT / 2 private val BIO_ACTION_RESERVE = 104.dp private val BIO_ACTION_HEIGHT = 48.dp -private val BIO_ACTION_OFFSET = 8.dp +private val BIO_ACTION_OFFSET = 16.dp +private val BIO_ACTION_END_OFFSET = 8.dp private val BIO_TOP_GAP = 8.dp -private val BIO_LINES_TOP_GAP = 8.dp private val BIO_BOTTOM_GAP = 8.dp internal fun estimateScrollPx( @@ -644,7 +644,7 @@ private fun ArtistBio( } } Box { - Column(Modifier.fillMaxWidth().padding(top = BIO_LINES_TOP_GAP)) { + Column(Modifier.fillMaxWidth()) { bioLines.lines.forEachIndexed { index, line -> val isControl = index == bioLines.lines.lastIndex && (bioLines.more || bioLines.less) Text( @@ -662,7 +662,11 @@ private fun ArtistBio( ) } } - Row(Modifier.align(Alignment.TopEnd).offset(y = -BIO_ACTION_OFFSET)) { + Row( + Modifier + .align(Alignment.TopEnd) + .offset(x = BIO_ACTION_END_OFFSET, y = -BIO_ACTION_OFFSET), + ) { ArtistActions(artist, starred, onToggleStar, onMore) } }