Expand message long-press hit area to entire cell (#6405)

* DS-031: Expand message long-press hit area to entire cell

Move the click modifier from the inner content Row (wrap-content) to the outer Box (full-width) so long-press can be triggered anywhere across the row, not only on the avatar+bubble area. Aligns the long-press surface with the swipe surface and matches iOS behaviour.

Adds a ripple indication so the gesture target is visible to the user.

* Extract message click predicates into named locals

Replace inline boolean expressions inside the click and long-click lambdas with `canOpenThread` and `canOpenActions`, so the gesture handlers read as intent rather than condition bookkeeping.

* Collapse message item Box+Row wrapper into a single Row

Replace the outer `Box(fillMaxWidth, contentAlignment)` + inner `Row(wrapContentWidth)` pair with a single `Row(fillMaxWidth, horizontalArrangement)`. The Box was only providing edge alignment, which a Row achieves directly via `horizontalArrangement` mapped from `MessageAlignment`.

Drops the now-redundant `Stream_MessageItem` test tag (zero references outside this file) and keeps `Stream_MessageCell` on the merged Row, where it accurately labels the message cell that the gesture surface lives on.

* Enable message clicks only when threads or actions are available

Author: andremion

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

@@ -24,6 +24,7 @@ import androidx.compose.foundation.background
 import androidx.compose.foundation.combinedClickable
 import androidx.compose.foundation.gestures.detectHorizontalDragGestures
 import androidx.compose.foundation.interaction.MutableInteractionSource
+import androidx.compose.foundation.layout.Arrangement
 import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.ColumnScope
@@ -38,7 +39,7 @@ import androidx.compose.foundation.layout.size
 import androidx.compose.foundation.layout.width
 import androidx.compose.foundation.layout.widthIn
 import androidx.compose.foundation.layout.wrapContentHeight
-import androidx.compose.foundation.layout.wrapContentWidth
+import androidx.compose.material3.ripple
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.LaunchedEffect
 import androidx.compose.runtime.getValue
@@ -174,12 +175,16 @@ public fun MessageContainer(
     val focusState = messageItem.focusState
     val haptic = LocalHapticFeedback.current
 
+    val canOpenThread = message.isThreadStart() && !messageItem.isInThread
+    val canOpenActions = !message.isDeleted() && !message.isUploading()
+
     val clickModifier = Modifier.combinedClickable(
         interactionSource = remember { MutableInteractionSource() },
-        indication = null,
-        onClick = { if (message.isThreadStart() && !messageItem.isInThread) onThreadClick(message) },
+        indication = ripple(),
+        enabled = canOpenThread || canOpenActions,
+        onClick = { if (canOpenThread) onThreadClick(message) },
         onLongClick = {
-            if (!message.isDeleted() && !message.isUploading()) {
+            if (canOpenActions) {
                 haptic.performHapticFeedback(HapticFeedbackType.LongPress)
                 onLongItemClick(message)
             }
@@ -218,89 +223,87 @@ public fun MessageContainer(
         onReply = { onReply(replyMessage) },
         isSwipeable = isSwipeable,
     ) {
-        Box(
+        Row(
             modifier = Modifier
-                .testTag("Stream_MessageItem")
+                .testTag("Stream_MessageCell")
                 .fillMaxWidth()
                 .wrapContentHeight()
                 .background(color = color)
+                .then(clickModifier)
                 .semantics { contentDescription = description },
-            contentAlignment = messageAlignment.itemAlignment,
+            horizontalArrangement = if (messageAlignment == MessageAlignment.Start) {
+                Arrangement.Start
+            } else {
+                Arrangement.End
+            },
         ) {
-            Row(
-                Modifier
-                    .wrapContentWidth()
-                    .then(clickModifier)
-                    .testTag("Stream_MessageCell"),
-            ) {
-                with(ChatTheme.componentFactory) {
-                    when (messageAlignment) {
-                        MessageAlignment.Start -> MessageAuthor(
-                            params = MessageAuthorParams(
-                                messageItem = messageItem,
-                                onUserAvatarClick = onUserAvatarClick,
-                            ),
-                        )
+            with(ChatTheme.componentFactory) {
+                when (messageAlignment) {
+                    MessageAlignment.Start -> MessageAuthor(
+                        params = MessageAuthorParams(
+                            messageItem = messageItem,
+                            onUserAvatarClick = onUserAvatarClick,
+                        ),
+                    )
 
-                        MessageAlignment.End -> MessageSpacer(params = MessageSpacerParams(messageItem))
-                    }
+                    MessageAlignment.End -> MessageSpacer(params = MessageSpacerParams(messageItem))
+                }
 
-                    Column(
-                        modifier = Modifier.weight(1f, fill = false),
-                        horizontalAlignment = messageAlignment.contentAlignment,
-                    ) {
-                        MessageTop(
-                            params = MessageTopParams(
-                                messageItem = messageItem,
-                                onThreadClick = onThreadClick,
-                            ),
-                        )
-                        MessageContentWithReactions(
-                            messageAlignment = messageAlignment,
-                            reactions = rememberMessageReactions(message)?.let { reactions ->
-                                {
-                                    MessageReactions(
-                                        params = MessageReactionsParams(
-                                            message = message,
-                                            reactions = reactions,
-                                            onClick = onReactionsClick,
-                                        ),
-                                    )
-                                }
-                            },
-                            content = {
-                                MessageContent(
-                                    params = MessageContentParams(
-                                        messageItem = messageItem,
-                                        onLongItemClick = onLongItemClick,
-                                        onMediaGalleryPreviewResult = onMediaGalleryPreviewResult,
-                                        onGiphyActionClick = onGiphyActionClick,
-                                        onQuotedMessageClick = onQuotedMessageClick,
-                                        onLinkClick = onLinkClick,
-                                        onUserMentionClick = onUserMentionClick,
-                                        onPollUpdated = onPollUpdated,
-                                        onCastVote = onCastVote,
-                                        onRemoveVote = onRemoveVote,
-                                        selectPoll = selectPoll,
-                                        onAddAnswer = onAddAnswer,
-                                        onClosePoll = onClosePoll,
-                                        onAddPollOption = onAddPollOption,
+                Column(
+                    modifier = Modifier.weight(1f, fill = false),
+                    horizontalAlignment = messageAlignment.contentAlignment,
+                ) {
+                    MessageTop(
+                        params = MessageTopParams(
+                            messageItem = messageItem,
+                            onThreadClick = onThreadClick,
+                        ),
+                    )
+                    MessageContentWithReactions(
+                        messageAlignment = messageAlignment,
+                        reactions = rememberMessageReactions(message)?.let { reactions ->
+                            {
+                                MessageReactions(
+                                    params = MessageReactionsParams(
+                                        message = message,
+                                        reactions = reactions,
+                                        onClick = onReactionsClick,
                                     ),
                                 )
-                            },
-                        )
-                        MessageBottom(params = MessageBottomParams(messageItem = messageItem))
-                    }
+                            }
+                        },
+                        content = {
+                            MessageContent(
+                                params = MessageContentParams(
+                                    messageItem = messageItem,
+                                    onLongItemClick = onLongItemClick,
+                                    onMediaGalleryPreviewResult = onMediaGalleryPreviewResult,
+                                    onGiphyActionClick = onGiphyActionClick,
+                                    onQuotedMessageClick = onQuotedMessageClick,
+                                    onLinkClick = onLinkClick,
+                                    onUserMentionClick = onUserMentionClick,
+                                    onPollUpdated = onPollUpdated,
+                                    onCastVote = onCastVote,
+                                    onRemoveVote = onRemoveVote,
+                                    selectPoll = selectPoll,
+                                    onAddAnswer = onAddAnswer,
+                                    onClosePoll = onClosePoll,
+                                    onAddPollOption = onAddPollOption,
+                                ),
+                            )
+                        },
+                    )
+                    MessageBottom(params = MessageBottomParams(messageItem = messageItem))
+                }
 
-                    when (messageAlignment) {
-                        MessageAlignment.Start -> MessageSpacer(params = MessageSpacerParams(messageItem))
-                        MessageAlignment.End -> MessageAuthor(
-                            params = MessageAuthorParams(
-                                messageItem = messageItem,
-                                onUserAvatarClick = onUserAvatarClick,
-                            ),
-                        )
-                    }
+                when (messageAlignment) {
+                    MessageAlignment.Start -> MessageSpacer(params = MessageSpacerParams(messageItem))
+                    MessageAlignment.End -> MessageAuthor(
+                        params = MessageAuthorParams(
+                            messageItem = messageItem,
+                            onUserAvatarClick = onUserAvatarClick,
+                        ),
+                    )
                 }
             }
         }