From c7483a4f0da9e2cba182a38d2ea4d8ba669dba53 Mon Sep 17 00:00:00 2001 From: nrobi144 Date: Thu, 28 May 2026 13:18:23 +0300 Subject: [PATCH] fix(desktop): SearchPill hover bounds + AccountSwitcher ripple size - SearchPill: move hoverHighlight() inside Surface content Row so it's clipped to the pill's 36dp height (was drawing on parent Row height) - AccountSwitcherDropdown: reduce IconButton from 48dp to 40dp so ripple circle fits within collapsed sidebar width Co-Authored-By: Claude Opus 4.6 (1M context) --- .../amethyst/desktop/ui/account/AccountSwitcherDropdown.kt | 2 +- .../vitorpamplona/amethyst/desktop/ui/search/SearchPill.kt | 5 ++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/desktopApp/src/jvmMain/kotlin/com/vitorpamplona/amethyst/desktop/ui/account/AccountSwitcherDropdown.kt b/desktopApp/src/jvmMain/kotlin/com/vitorpamplona/amethyst/desktop/ui/account/AccountSwitcherDropdown.kt index 03bff1775f..95eff2e687 100644 --- a/desktopApp/src/jvmMain/kotlin/com/vitorpamplona/amethyst/desktop/ui/account/AccountSwitcherDropdown.kt +++ b/desktopApp/src/jvmMain/kotlin/com/vitorpamplona/amethyst/desktop/ui/account/AccountSwitcherDropdown.kt @@ -99,7 +99,7 @@ fun AccountSwitcherDropdown( Box(modifier = modifier) { IconButton( onClick = { expanded = true }, - modifier = Modifier.size(48.dp), + modifier = Modifier.size(40.dp), ) { Icon( MaterialSymbols.Person, diff --git a/desktopApp/src/jvmMain/kotlin/com/vitorpamplona/amethyst/desktop/ui/search/SearchPill.kt b/desktopApp/src/jvmMain/kotlin/com/vitorpamplona/amethyst/desktop/ui/search/SearchPill.kt index 71a51f668b..c2751a6a04 100644 --- a/desktopApp/src/jvmMain/kotlin/com/vitorpamplona/amethyst/desktop/ui/search/SearchPill.kt +++ b/desktopApp/src/jvmMain/kotlin/com/vitorpamplona/amethyst/desktop/ui/search/SearchPill.kt @@ -33,7 +33,6 @@ import androidx.compose.material3.Text import androidx.compose.runtime.Composable import androidx.compose.ui.Alignment import androidx.compose.ui.Modifier -import androidx.compose.ui.draw.clip import androidx.compose.ui.unit.dp import com.vitorpamplona.amethyst.commons.icons.symbols.Icon import com.vitorpamplona.amethyst.commons.icons.symbols.MaterialSymbols @@ -50,11 +49,11 @@ fun SearchPill( onClick = onClick, shape = RoundedCornerShape(999.dp), color = MaterialTheme.colorScheme.surfaceContainerHigh, - modifier = modifier.height(36.dp).clip(RoundedCornerShape(999.dp)).hoverHighlight(), + modifier = modifier.height(36.dp), ) { Row( verticalAlignment = Alignment.CenterVertically, - modifier = Modifier.padding(horizontal = 12.dp), + modifier = Modifier.hoverHighlight().padding(horizontal = 12.dp), ) { Icon( MaterialSymbols.Search,