From e45fbccc4eeb8f4242b207f1292a9e287a875ea9 Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 18 Jun 2026 20:28:27 +0000 Subject: [PATCH] style: match collapsed reply avatar spacing to NoteComposeLayout Drop the oversized 55dp centering box that left an extra gap between the avatar and the name. Use NoteComposeLayout's own metrics instead: 12dp leading padding and a 10dp avatar-to-name gap, so the spacing around the collapsed avatar matches an expanded note. Co-Authored-By: Claude Opus 4.8 (1M context) Claude-Session: https://claude.ai/code/session_015hwQiQbgboo8LScPdDHDJn --- .../ui/screen/loggedIn/threadview/ThreadFeedView.kt | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/amethyst/src/main/java/com/vitorpamplona/amethyst/ui/screen/loggedIn/threadview/ThreadFeedView.kt b/amethyst/src/main/java/com/vitorpamplona/amethyst/ui/screen/loggedIn/threadview/ThreadFeedView.kt index e71589001e..d96d904fe8 100644 --- a/amethyst/src/main/java/com/vitorpamplona/amethyst/ui/screen/loggedIn/threadview/ThreadFeedView.kt +++ b/amethyst/src/main/java/com/vitorpamplona/amethyst/ui/screen/loggedIn/threadview/ThreadFeedView.kt @@ -27,7 +27,6 @@ import androidx.compose.foundation.ExperimentalFoundationApi import androidx.compose.foundation.clickable import androidx.compose.foundation.combinedClickable import androidx.compose.foundation.layout.Arrangement -import androidx.compose.foundation.layout.Box import androidx.compose.foundation.layout.Column import androidx.compose.foundation.layout.PaddingValues import androidx.compose.foundation.layout.Row @@ -546,16 +545,14 @@ private fun CollapsedNoteCompose( modifier .fillMaxWidth() .clickable(onClick = onExpand) - .padding(horizontal = 10.dp, vertical = 10.dp), + // Match NoteComposeLayout: 12dp leading/trailing padding, 10dp top. + .padding(start = 12.dp, end = 12.dp, top = 10.dp, bottom = 10.dp), verticalAlignment = Alignment.CenterVertically, ) { - // Center the smaller avatar inside the full 55dp slot so the text lines up with the - // expanded NoteCompose, which uses a 55dp avatar. - Box(modifier = Modifier.width(Size55dp), contentAlignment = Alignment.Center) { - NoteAuthorPicture(baseNote = baseNote, size = Size40dp, accountViewModel = accountViewModel, nav = nav) - } + NoteAuthorPicture(baseNote = baseNote, size = Size40dp, accountViewModel = accountViewModel, nav = nav) - Spacer(modifier = StdHorzSpacer) + // 10dp gap between the avatar and the name, same as NoteComposeLayout's authorContentGap. + Spacer(modifier = Modifier.width(10.dp)) Column(modifier = Modifier.weight(1f)) { NoteUsernameDisplay(baseNote, accountViewModel = accountViewModel)