laantungir/amethyst
1
0
Fork 0
mirror of https://github.com/vitorpamplona/amethyst.git synced 2026-08-10 00:16:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: make notify chip row spacing match horizontal spacing

Browse Source 
Chips render through a selectable Surface that enforces a 48dp minimum
touch target, inflating each chip's measured height above its visible 32dp
pill. That invisible padding dominated the gap between wrapped rows, so the
previous verticalArrangement bump was imperceptible. Disable the minimum
interactive size around the chips so they measure at their visible height
and the row spacing matches the 6dp horizontal spacing.
This commit is contained in:
Claude
2026-06-13 14:48:50 +00:00
parent 45ce6f2815
commit 6fcbecb293
1 changed files with 16 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
amethyst/src/main/java/com/vitorpamplona/amethyst/ui/note/creators/notify/Notifying.kt
+16 -5
View File
@@ -29,12 +29,15 @@ import androidx.compose.material3.AssistChip
import androidx.compose.material3.AssistChipDefaults
import androidx.compose.material3.InputChip
import androidx.compose.material3.InputChipDefaults
import androidx.compose.material3.LocalMinimumInteractiveComponentSize
import androidx.compose.material3.MaterialTheme
import androidx.compose.material3.Text
import androidx.compose.runtime.Composable
import androidx.compose.runtime.CompositionLocalProvider
import androidx.compose.ui.Alignment.Companion.CenterVertically
import androidx.compose.ui.Modifier
import androidx.compose.ui.text.font.FontWeight
import androidx.compose.ui.unit.Dp
import androidx.compose.ui.unit.dp
import com.vitorpamplona.amethyst.R
import com.vitorpamplona.amethyst.commons.icons.symbols.Icon
@@ -72,12 +75,20 @@ fun Notifying(
modifier = Modifier.align(CenterVertically),
)
mentions?.forEach { user ->
NotifyUserChip(user, accountViewModel) { onClick(user) }
}
// The chips render through a selectable Surface that enforces a 48dp minimum
// touch target, inflating each chip's measured height well above its visible
// 32dp pill. That invisible padding would dominate the gap between wrapped
// rows and make verticalArrangement barely noticeable. Disabling the minimum
// interactive size lets the chips measure at their visible height so the row
// spacing matches the horizontal spacing between chips.
CompositionLocalProvider(LocalMinimumInteractiveComponentSize provides Dp.Unspecified) {
mentions?.forEach { user ->
NotifyUserChip(user, accountViewModel) { onClick(user) }
}
if (onAddUser != null) {
AddUserChip(onAddUser)
if (onAddUser != null) {
AddUserChip(onAddUser)
}
}
}
}