From 4af904bf2ac909281329ddeab790800fcaa0981b Mon Sep 17 00:00:00 2001 From: Jeena Date: Tue, 13 Oct 2026 07:45:17 +0000 Subject: [PATCH] home: Nudge the logo below the text baseline On one baseline the glyph looked like it was sinking into the title; 1dp lower matches how the launcher icon sits next to text. Closes #11 --- .../net/jeena/jtubetv/ui/screens/HomeScreen.kt | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/app/src/main/java/net/jeena/jtubetv/ui/screens/HomeScreen.kt b/app/src/main/java/net/jeena/jtubetv/ui/screens/HomeScreen.kt index cd1c618..432217c 100644 --- a/app/src/main/java/net/jeena/jtubetv/ui/screens/HomeScreen.kt +++ b/app/src/main/java/net/jeena/jtubetv/ui/screens/HomeScreen.kt @@ -152,9 +152,8 @@ fun HomeScreen(viewModel: AppViewModel = androidx.lifecycle.viewmodel.compose.vi }, ) { Column { - // Persistent top bar: logo/title left, account button right. + // Persistent top bar: logo/title left, account indicator right. TopBar( - onAccountClick = { drawerOpen = !drawerOpen }, profile = userProfile, ) androidx.compose.runtime.key(epoch) { @@ -279,13 +278,11 @@ private fun Destination.icon(): ImageVector = */ @Composable private fun TopBar( - onAccountClick: () -> Unit, profile: PeerTubeRepository.UserProfile?, modifier: Modifier = Modifier, ) { Row( - modifier = modifier - .fillMaxWidth() + modifier = Modifier.fillMaxWidth() .padding(start = 24.dp, end = 24.dp, top = 16.dp, bottom = 8.dp), verticalAlignment = Alignment.CenterVertically, ) { @@ -297,7 +294,7 @@ private fun TopBar( Box( modifier = Modifier .align(Alignment.Bottom) - .padding(bottom = 17.dp) // slightly below the text baseline + .padding(bottom = 16.dp) // sit the logo a touch below the text baseline .size(32.dp) // match the avatar row so both headers align .clip(RoundedCornerShape(8.dp)) .background(Color(0xFF101010)), @@ -320,11 +317,10 @@ private fun TopBar( style = androidx.compose.material3.MaterialTheme.typography.titleLarge, ) Spacer(modifier = Modifier.weight(1f)) - // Account indicator: avatar + display name. Purely visual on TV (D-pad - // focus and activation stay in the content/drawer); a click (mouse/touch) - // still opens the menu drawer. + // Account indicator: avatar + display name. Purely visual — TV focus and + // activation stay in the content/drawer (issue #7's "all the way up" + // surprise must not return); the drawer is reached via the LEFT edge. Row( - modifier = Modifier.clickable(onClick = onAccountClick), verticalAlignment = Alignment.CenterVertically, horizontalArrangement = androidx.compose.foundation.layout.Arrangement.spacedBy(8.dp), ) {