fix: increase bottom bar icon size to compensate for MaterialSymbols padding
MaterialSymbols glyphs have more internal padding than the previous MaterialIcons, so at 20dp they looked smaller than before. Bump the NavigationBar icons to 24dp (Material's default NavigationBar icon size) and widen the notification-dot box accordingly to preserve its offset.
This commit is contained in:
+4
-4
@@ -51,8 +51,8 @@ import com.vitorpamplona.amethyst.ui.stringRes
|
||||
import com.vitorpamplona.amethyst.ui.theme.DividerThickness
|
||||
import com.vitorpamplona.amethyst.ui.theme.Size0dp
|
||||
import com.vitorpamplona.amethyst.ui.theme.Size10Modifier
|
||||
import com.vitorpamplona.amethyst.ui.theme.Size20dp
|
||||
import com.vitorpamplona.amethyst.ui.theme.Size23dp
|
||||
import com.vitorpamplona.amethyst.ui.theme.Size24dp
|
||||
import com.vitorpamplona.amethyst.ui.theme.Size27dp
|
||||
|
||||
@Composable
|
||||
fun AppBottomBar(
|
||||
@@ -141,8 +141,8 @@ private fun NotifiableIcon(
|
||||
destination: Route,
|
||||
accountViewModel: AccountViewModel,
|
||||
) {
|
||||
Box(Modifier.size(Size23dp)) {
|
||||
val iconSizeModifier = Modifier.size(Size20dp)
|
||||
Box(Modifier.size(Size27dp)) {
|
||||
val iconSizeModifier = Modifier.size(Size24dp)
|
||||
val description = stringRes(def.labelRes)
|
||||
Icon(
|
||||
symbol = def.icon,
|
||||
|
||||
@@ -119,6 +119,7 @@ val Size22dp = 22.dp
|
||||
val Size23dp = 23.dp
|
||||
val Size24dp = 24.dp
|
||||
val Size25dp = 25.dp
|
||||
val Size27dp = 27.dp
|
||||
val Size30dp = 30.dp
|
||||
val Size34dp = 34.dp
|
||||
val Size35dp = 35.dp
|
||||
|
||||
Reference in New Issue
Block a user