CMM-885 support HE attachments

WordPress/src/main/java/org/wordpress/android/support/he/ui/HEConversationDetailScreen.kt

@@ -1,6 +1,7 @@
 package org.wordpress.android.support.he.ui
 
 import android.content.res.Configuration.UI_MODE_NIGHT_YES
+import android.net.Uri
 import androidx.compose.foundation.background
 import androidx.compose.foundation.layout.Arrangement
 import androidx.compose.foundation.layout.Box
@@ -10,28 +11,23 @@ import androidx.compose.foundation.layout.Spacer
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.fillMaxWidth
 import androidx.compose.foundation.layout.height
-import androidx.compose.foundation.layout.imePadding
 import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.layout.size
 import androidx.compose.foundation.lazy.LazyColumn
 import androidx.compose.foundation.lazy.items
 import androidx.compose.foundation.lazy.rememberLazyListState
-import androidx.compose.foundation.rememberScrollState
 import androidx.compose.foundation.shape.RoundedCornerShape
-import androidx.compose.foundation.verticalScroll
 import androidx.compose.material.icons.Icons
 import androidx.compose.material.icons.automirrored.filled.Reply
 import androidx.compose.material3.Button
 import androidx.compose.material3.CircularProgressIndicator
 import androidx.compose.material3.ExperimentalMaterial3Api
 import androidx.compose.material3.Icon
 import androidx.compose.material3.MaterialTheme
-import androidx.compose.material3.ModalBottomSheet
 import androidx.compose.material3.Scaffold
 import androidx.compose.material3.SnackbarHost
 import androidx.compose.material3.SnackbarHostState
 import androidx.compose.material3.Text
-import androidx.compose.material3.TextButton
 import androidx.compose.material3.rememberModalBottomSheetState
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.LaunchedEffect
@@ -40,6 +36,7 @@ import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
 import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.runtime.setValue
+import kotlinx.coroutines.launch
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.platform.LocalResources
@@ -52,11 +49,11 @@ import androidx.compose.ui.semantics.semantics
 import androidx.compose.ui.text.font.FontWeight
 import androidx.compose.ui.tooling.preview.Preview
 import androidx.compose.ui.unit.dp
-import kotlinx.coroutines.launch
 import org.wordpress.android.R
 import org.wordpress.android.support.aibot.util.formatRelativeTime
 import org.wordpress.android.support.he.model.SupportConversation
 import org.wordpress.android.support.he.model.SupportMessage
+import org.wordpress.android.support.he.util.AttachmentActionsListener
 import org.wordpress.android.support.he.util.generateSampleHESupportConversations
 import org.wordpress.android.ui.compose.components.MainTopAppBar
 import org.wordpress.android.ui.compose.components.NavigationIcons
@@ -72,7 +69,9 @@ fun HEConversationDetailScreen(
     messageSendResult: HESupportViewModel.MessageSendResult? = null,
     onBackClick: () -> Unit,
     onSendMessage: (message: String, includeAppLogs: Boolean) -> Unit,
-    onClearMessageSendResult: () -> Unit = {}
+    onClearMessageSendResult: () -> Unit = {},
+    attachments: List<Uri> = emptyList(),
+    attachmentActionsListener: AttachmentActionsListener,
 ) {
     val listState = rememberLazyListState()
     val sheetState = rememberModalBottomSheetState(skipPartiallyExpanded = true)
@@ -139,7 +138,7 @@ fun HEConversationDetailScreen(
             ) { message ->
                 MessageItem(
                     message = message,
-                    timestamp = formatRelativeTime(message.createdAt, resources)
+                    timestamp = formatRelativeTime(message.createdAt, resources),
                 )
             }
 
@@ -157,7 +156,7 @@ fun HEConversationDetailScreen(
     }
 
     if (showBottomSheet) {
-        ReplyBottomSheet(
+        HEConversationReplyBottomSheet(
             sheetState = sheetState,
             isSending = isSendingMessage,
             messageSendResult = messageSendResult,
@@ -181,7 +180,9 @@ fun HEConversationDetailScreen(
                 draftMessageText = ""
                 draftIncludeAppLogs = false
                 onClearMessageSendResult()
-            }
+            },
+            attachments = attachments,
+            attachmentActionsListener = attachmentActionsListener
         )
     }
 }
@@ -351,106 +352,6 @@ private fun ReplyButton(
     }
 }
 
-@OptIn(ExperimentalMaterial3Api::class)
-@Composable
-private fun ReplyBottomSheet(
-    sheetState: androidx.compose.material3.SheetState,
-    isSending: Boolean = false,
-    messageSendResult: HESupportViewModel.MessageSendResult? = null,
-    initialMessageText: String = "",
-    initialIncludeAppLogs: Boolean = false,
-    onDismiss: (currentMessage: String, currentIncludeAppLogs: Boolean) -> Unit,
-    onSend: (String, Boolean) -> Unit,
-    onMessageSentSuccessfully: () -> Unit
-) {
-    var messageText by remember { mutableStateOf(initialMessageText) }
-    var includeAppLogs by remember { mutableStateOf(initialIncludeAppLogs) }
-    val scrollState = rememberScrollState()
-
-    // Close the sheet when sending completes successfully
-    LaunchedEffect(messageSendResult) {
-        when (messageSendResult) {
-            is HESupportViewModel.MessageSendResult.Success -> {
-                // Message sent successfully, close the sheet and clear draft
-                onDismiss("", false)
-                onMessageSentSuccessfully()
-            }
-            is HESupportViewModel.MessageSendResult.Failure -> {
-                // Message failed to send, draft is saved onDismiss
-                // The error will be shown via snackbar from the Activity
-                onDismiss("", false)
-            }
-            null -> {
-                // No result yet, do nothing
-            }
-        }
-    }
-
-    ModalBottomSheet(
-        onDismissRequest = { onDismiss(messageText, includeAppLogs) },
-        sheetState = sheetState
-    ) {
-        Column(
-            modifier = Modifier
-                .fillMaxWidth()
-                .imePadding()
-                .verticalScroll(scrollState)
-                .padding(horizontal = 16.dp)
-                .padding(bottom = 32.dp)
-        ) {
-            Row(
-                modifier = Modifier
-                    .fillMaxWidth()
-                    .padding(bottom = 24.dp),
-                horizontalArrangement = Arrangement.SpaceBetween,
-                verticalAlignment = Alignment.CenterVertically
-            ) {
-                TextButton(
-                    onClick = { onDismiss(messageText, includeAppLogs) },
-                    enabled = !isSending
-                ) {
-                    Text(
-                        text = stringResource(R.string.cancel),
-                        style = MaterialTheme.typography.titleMedium
-                    )
-                }
-
-                Text(
-                    text = stringResource(R.string.he_support_reply_button),
-                    style = MaterialTheme.typography.titleLarge,
-                    fontWeight = FontWeight.Bold,
-                    modifier = Modifier.semantics { heading() }
-                )
-
-                TextButton(
-                    onClick = { onSend(messageText, includeAppLogs) },
-                    enabled = messageText.isNotBlank() && !isSending
-                ) {
-                    if (isSending) {
-                        CircularProgressIndicator(
-                            modifier = Modifier.size(20.dp),
-                            strokeWidth = 2.dp
-                        )
-                    } else {
-                        Text(
-                            text = stringResource(R.string.he_support_send_button),
-                            style = MaterialTheme.typography.titleMedium
-                        )
-                    }
-                }
-            }
-
-            TicketMainContentView(
-                messageText = messageText,
-                includeAppLogs = includeAppLogs,
-                onMessageChanged = { message -> messageText = message },
-                onIncludeAppLogsChanged = { checked -> includeAppLogs = checked },
-                enabled = !isSending
-            )
-        }
-    }
-}
-
 @Preview(showBackground = true, name = "HE Conversation Detail")
 @Composable
 private fun HEConversationDetailScreenPreview() {
@@ -462,7 +363,15 @@ private fun HEConversationDetailScreenPreview() {
             snackbarHostState = snackbarHostState,
             conversation = sampleConversation,
             onBackClick = { },
-            onSendMessage = { _, _ -> }
+            onSendMessage = { _, _ -> },
+            attachmentActionsListener = object : AttachmentActionsListener {
+                override fun onAddImageClick() {
+                    // stub
+                }
+                override fun onRemoveImage(uri: Uri) {
+                    // stub
+                }
+            }
         )
     }
 }
@@ -478,7 +387,15 @@ private fun HEConversationDetailScreenPreviewDark() {
             snackbarHostState = snackbarHostState,
             conversation = sampleConversation,
             onBackClick = { },
-            onSendMessage = { _, _ -> }
+            onSendMessage = { _, _ -> },
+            attachmentActionsListener = object : AttachmentActionsListener {
+                override fun onAddImageClick() {
+                    // stub
+                }
+                override fun onRemoveImage(uri: Uri) {
+                    // stub
+                }
+            }
         )
     }
 }
@@ -494,7 +411,15 @@ private fun HEConversationDetailScreenWordPressPreview() {
             snackbarHostState = snackbarHostState,
             conversation = sampleConversation,
             onBackClick = { },
-            onSendMessage = { _, _ -> }
+            onSendMessage = { _, _ -> },
+            attachmentActionsListener = object : AttachmentActionsListener {
+                override fun onAddImageClick() {
+                    // stub
+                }
+                override fun onRemoveImage(uri: Uri) {
+                    // stub
+                }
+            }
         )
     }
 }
@@ -511,7 +436,15 @@ private fun HEConversationDetailScreenPreviewWordPressDark() {
             isLoading = true,
             conversation = sampleConversation,
             onBackClick = { },
-            onSendMessage = { _, _ -> }
+            onSendMessage = { _, _ -> },
+            attachmentActionsListener = object : AttachmentActionsListener {
+                override fun onAddImageClick() {
+                    // stub
+                }
+                override fun onRemoveImage(uri: Uri) {
+                    // stub
+                }
+            }
         )
     }
 }