fix: message item long tap action [WPB-5952] (#2553)

wireapp · Dec 22, 2023 · 069d619 · 069d619
1 parent 8d01df7
commit 069d619
Showing 1 changed file with 13 additions and 13 deletions.
diff --git a/app/src/main/kotlin/com/wire/android/ui/home/conversations/MessageItem.kt b/app/src/main/kotlin/com/wire/android/ui/home/conversations/MessageItem.kt
@@ -25,7 +25,6 @@ import androidx.compose.animation.animateColorAsState
 import androidx.compose.animation.core.tween
 import androidx.compose.foundation.ExperimentalFoundationApi
 import androidx.compose.foundation.background
-import androidx.compose.foundation.clickable
 import androidx.compose.foundation.combinedClickable
 import androidx.compose.foundation.layout.Arrangement
 import androidx.compose.foundation.layout.Box
@@ -168,24 +167,25 @@ fun MessageItem(
 
         Box(
             backgroundColorModifier
-                .clickable(enabled = isContentClickable, onClick = {
-                    onMessageClick(message.header.messageId)
-                })
+                .combinedClickable(enabled = true, onClick = {
+                    if (isContentClickable) {
+                        onMessageClick(message.header.messageId)
+                    }
+                },
+                    onLongClick = remember(message) {
+                        {
+                            if (!isContentClickable) {
+                                onLongClicked(message)
+                            }
+                        }
+                    }
+                )
         ) {
             // padding needed to have same top padding for avatar and rest composables in message item
             val fullAvatarOuterPadding = dimensions().avatarClickablePadding + dimensions().avatarStatusBorderSize
             Row(
                 Modifier
                     .fillMaxWidth()
-                    .apply {
-                        if (!isContentClickable) {
-                            combinedClickable(
-                                enabled = message.isAvailable,
-                                onClick = { },
-                                onLongClick = remember(message) { { onLongClicked(message) } }
-                            )
-                        }
-                    }
                     .padding(
                         end = dimensions().messageItemHorizontalPadding,
                         top = if (showAuthor) dimensions().spacing0x else dimensions().spacing4x,