From 3ebf59dd21cbf9fb3669bd0de8a894f240e327e2 Mon Sep 17 00:00:00 2001
From: austinried <4966622+austinried@users.noreply.github.com>
Date: Thu, 24 Sep 2026 07:30:13 +0000
Subject: [PATCH] clamp the now playing title and subtitle heights

---
 .../subtracks/ui/playback/NowPlayingScreen.kt | 25 +++++++++++--------
 1 file changed, 14 insertions(+), 11 deletions(-)

diff --git a/app/src/main/kotlin/com/subtracks/ui/playback/NowPlayingScreen.kt b/app/src/main/kotlin/com/subtracks/ui/playback/NowPlayingScreen.kt
index a46f0c30..641c28b2 100644
--- a/app/src/main/kotlin/com/subtracks/ui/playback/NowPlayingScreen.kt
+++ b/app/src/main/kotlin/com/subtracks/ui/playback/NowPlayingScreen.kt
@@ -195,33 +195,36 @@ fun NowPlayingScreen(
                             )
                         }
                         val density = LocalDensity.current
+                        val titleHeight =
+                            with(density) {
+                                MaterialTheme.typography.headlineSmall.lineHeight
+                                    .toDp()
+                            }
+                        val subtitleHeight =
+                            with(density) {
+                                MaterialTheme.typography.bodyMedium.lineHeight
+                                    .toDp()
+                            }
                         Column(
                             horizontalAlignment = Alignment.CenterHorizontally,
+                            verticalArrangement = Arrangement.spacedBy(6.dp),
                             modifier =
                                 Modifier
                                     .padding(top = 28.dp)
-                                    .height(
-                                        with(density) {
-                                            MaterialTheme.typography.headlineSmall.lineHeight
-                                                .toDp() +
-                                                6.dp +
-                                                MaterialTheme.typography.bodyMedium.lineHeight
-                                                    .toDp()
-                                        },
-                                    ),
+                                    .height(titleHeight + 6.dp + subtitleHeight),
                         ) {
                             Text(
                                 text = state.item?.title.orEmpty(),
                                 style = titleStyle,
                                 maxLines = 1,
-                                modifier = Modifier.basicMarquee(),
+                                modifier = Modifier.height(titleHeight).basicMarquee(),
                             )
                             Text(
                                 text = listOfNotNull(state.item?.artist, state.item?.album).joinToString(" • "),
                                 style = subtitleStyle,
                                 color = MaterialTheme.colorScheme.onSurfaceVariant,
                                 maxLines = 1,
-                                modifier = Modifier.padding(top = 6.dp).basicMarquee(),
+                                modifier = Modifier.height(subtitleHeight).basicMarquee(),
                             )
                         }
                         SeekBar(