From f1397d29386f5427def4f890e2eb57a561d8f35b Mon Sep 17 00:00:00 2001 From: Alex Hart Date: Mon, 17 Aug 2026 14:04:00 -0300 Subject: [PATCH] Restore the media-send edit screen send button to 48dp. --- .../org/signal/mediasend/screens/edit/AddAMessageRow.kt | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/feature/media-send/src/main/java/org/signal/mediasend/screens/edit/AddAMessageRow.kt b/feature/media-send/src/main/java/org/signal/mediasend/screens/edit/AddAMessageRow.kt index 5113bfc83e..c16d028a96 100644 --- a/feature/media-send/src/main/java/org/signal/mediasend/screens/edit/AddAMessageRow.kt +++ b/feature/media-send/src/main/java/org/signal/mediasend/screens/edit/AddAMessageRow.kt @@ -40,6 +40,13 @@ import org.signal.mediasend.test.TestTags /** Mirrors the legacy send button's disabled tint, which has no equivalent in the core-ui color scheme. */ private val DisabledNextButtonColor = Color(0xFF777777) +/** + * Mirrors the legacy send button's size. Has to be stated rather than left to the [IconButtons.IconButton] default of + * 40dp: the default draws the container inside the 48dp of layout that [androidx.compose.material3.minimumInteractiveComponentSize] + * reserves, leaving a button that is 8dp smaller than it looks like it should be and smaller than the row it sits in. + */ +private val NextButtonSize = 48.dp + /** * Because we need to be able to support stuff like mentions, styled text, and custom emoji, we need to allow * the users of this feature to inject their own text-field. @@ -142,6 +149,7 @@ fun AddAMessageRow( IconButtons.IconButton( enabled = enabled, onClick = onNextClick, + size = NextButtonSize, onLongClick = if (canScheduleSend) scheduleSendMenuController::show else null, onLongClickLabel = stringResource(R.string.AddAMessageRow__schedule_send), colors = IconButtons.iconButtonColors(