Skip to content

Commit 3bb700f

Browse files
committed
Drop redundant outer padding around composer input row
The Row's padding(vertical = spacing2xs) added 8 dp to the composer surface with no Figma backing; the inner padding already provides the spec spacing. Also remove a wrapContentHeight() from the MessageComposer call in ChannelScreen — Column already grants intrinsic vertical sizing to its children, so the modifier had no effect. Re-record MessageComposerTest snapshots.
1 parent 140dceb commit 3bb700f

12 files changed

Lines changed: 2 additions & 5 deletions

stream-chat-android-compose/src/main/java/io/getstream/chat/android/compose/ui/messages/ChannelScreen.kt

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,6 @@ import androidx.compose.foundation.layout.fillMaxWidth
3131
import androidx.compose.foundation.layout.ime
3232
import androidx.compose.foundation.layout.padding
3333
import androidx.compose.foundation.layout.safeDrawingPadding
34-
import androidx.compose.foundation.layout.wrapContentHeight
3534
import androidx.compose.material3.MaterialTheme
3635
import androidx.compose.material3.Scaffold
3736
import androidx.compose.material3.SnackbarDuration
@@ -367,9 +366,7 @@ internal fun DefaultBottomBarContent(
367366

368367
Column {
369368
MessageComposer(
370-
modifier = Modifier
371-
.fillMaxWidth()
372-
.wrapContentHeight(),
369+
modifier = Modifier.fillMaxWidth(),
373370
viewModel = composerViewModel,
374371
isAttachmentPickerVisible = attachmentsPickerViewModel.isPickerVisible,
375372
onAttachmentsClick = attachmentsPickerViewModel::togglePickerVisibility,

stream-chat-android-compose/src/main/java/io/getstream/chat/android/compose/ui/messages/composer/MessageComposer.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,7 @@ import io.getstream.chat.android.compose.ui.theme.MessageComposerParams
6060
import io.getstream.chat.android.compose.ui.theme.MessageComposerTrailingContentParams
6161
import io.getstream.chat.android.compose.ui.theme.StreamTokens
6262
import io.getstream.chat.android.compose.ui.util.SnackbarPopup
63+
import io.getstream.chat.android.compose.ui.util.StreamSnackbar
6364
import io.getstream.chat.android.compose.ui.util.StreamSnackbarVariant
6465
import io.getstream.chat.android.compose.ui.util.StreamSnackbarVisuals
6566
import io.getstream.chat.android.compose.util.extensions.toSet
@@ -299,7 +300,6 @@ public fun MessageComposer(
299300

300301
Row(
301302
modifier = Modifier
302-
.padding(vertical = StreamTokens.spacing2xs)
303303
.fillMaxWidth()
304304
.padding(
305305
start = StreamTokens.spacingMd,
Loading
Loading
Loading
Loading
Loading
Loading
Loading
Loading

0 commit comments

Comments
 (0)