feat(Drive): Show loader when opening a file from conversation (WPB-28575)

app/src/main/kotlin/com/wire/android/ui/common/attachmentdraft/ui/FileHeaderView.kt

@@ -18,6 +18,7 @@
 package com.wire.android.ui.common.attachmentdraft.ui
 
 import androidx.compose.foundation.Image
+import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.Arrangement
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.Row
@@ -57,6 +58,8 @@ fun FileHeaderView(
     label: String? = null,
     labelColor: Color? = null,
     isError: Boolean = false,
+    showLabelInBubble: Boolean = false,
+    leadingContent: (@Composable () -> Unit)? = null,
 ) {
     val context = LocalContext.current
     val attachmentFileType = type ?: remember(extension) { AttachmentFileType.fromExtension(extension) }
@@ -67,31 +70,37 @@ fun FileHeaderView(
         verticalAlignment = Alignment.CenterVertically,
         horizontalArrangement = Arrangement.spacedBy(dimensions().spacing4x)
     ) {
-        if (isError) {
-            Icon(
-                modifier = Modifier.size(dimensions().spacing16x),
-                painter = painterResource(R.drawable.ic_warning_amber),
-                tint = colorsScheme().error,
-                contentDescription = null,
-            )
+        if (leadingContent != null) {
+            Box(modifier = Modifier.size(dimensions().spacing16x), contentAlignment = Alignment.Center) {
+                leadingContent()
+            }
         } else {
-            Image(
-                modifier = Modifier.size(dimensions().spacing16x),
-                painter = painterResource(id = attachmentFileType.icon()),
-                contentDescription = null,
-            )
+            if (isError) {
+                Icon(
+                    modifier = Modifier.size(dimensions().spacing16x),
+                    painter = painterResource(R.drawable.ic_warning_amber),
+                    tint = colorsScheme().error,
+                    contentDescription = null,
+                )
+            } else {
+                Image(
+                    modifier = Modifier.size(dimensions().spacing16x),
+                    painter = painterResource(id = attachmentFileType.icon()),
+                    contentDescription = null,
+                )
+            }
         }
         Text(
             text = "${extension.uppercase()} ($sizeString)",
             style = typography().subline01,
             color = messageStyle.textColor(),
         )
         Spacer(modifier = Modifier.weight(1f))
-        if (!messageStyle.isBubble()) {
+        if (!messageStyle.isBubble() || showLabelInBubble) {
             label?.let {
                 Text(
                     text = label,
-                    style = typography().body02,
+                    style = typography().subline01,
                     color = labelColor ?: messageStyle.textColor(),
                 )
             }
@@ -116,13 +125,11 @@ private fun PreviewFileHeader() {
             FileHeaderView(
                 extension = "DOCX",
                 size = 6796203,
-                label = "Downloading...",
                 messageStyle = MessageStyle.NORMAL
             )
             FileHeaderView(
                 extension = "ZIP",
                 size = 512746,
-                label = "Tap to view",
                 messageStyle = MessageStyle.NORMAL
             )
             FileHeaderView(

app/src/main/kotlin/com/wire/android/ui/common/multipart/MultipartAttachmentUi.kt

@@ -41,8 +41,15 @@ data class MultipartAttachmentUi(
     val progress: Float? = null,
     val isEditSupported: Boolean = false,
     val isAvailableOffline: Boolean = false,
+    val openLoadState: MultipartAttachmentOpenLoadState? = null,
 )
 
+sealed interface MultipartAttachmentOpenLoadState {
+    data class Loading(val progress: Float? = null) : MultipartAttachmentOpenLoadState
+    data class Ready(val localPath: String) : MultipartAttachmentOpenLoadState
+    data object Error : MultipartAttachmentOpenLoadState
+}
+
 enum class AssetSource {
     CELL, ASSET_STORAGE
 }

app/src/main/kotlin/com/wire/android/ui/home/conversations/model/messagetypes/multipart/MultipartAttachmentsView.kt

@@ -18,15 +18,15 @@
 package com.wire.android.ui.home.conversations.model.messagetypes.multipart
 
 import android.content.res.Configuration
+import android.widget.Toast
 import androidx.compose.foundation.layout.Arrangement
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.heightIn
 import androidx.compose.foundation.lazy.grid.GridCells
 import androidx.compose.foundation.lazy.grid.LazyVerticalGrid
 import androidx.compose.foundation.lazy.grid.items
 import androidx.compose.runtime.Composable
-import androidx.compose.runtime.getValue
-import androidx.compose.runtime.remember
+import androidx.compose.runtime.LaunchedEffect
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.layout.onVisibilityChanged
 import androidx.compose.ui.platform.LocalConfiguration
@@ -38,15 +38,18 @@ import coil3.request.ImageRequest
 import coil3.request.crossfade
 import com.wire.android.ui.common.colorsScheme
 import com.wire.android.ui.common.dimensions
+import com.wire.android.ui.common.multipart.MultipartAttachmentOpenLoadState
 import com.wire.android.ui.common.multipart.MultipartAttachmentUi
-import com.wire.android.ui.home.conversations.multipartAttachmentsViewModel
+import com.wire.android.ui.common.multipart.toUiModel
 import com.wire.android.ui.home.conversations.messages.item.MessageStyle
 import com.wire.android.ui.home.conversations.model.messagetypes.multipart.grid.AssetGridPreview
 import com.wire.android.ui.home.conversations.model.messagetypes.multipart.standalone.AssetPreview
+import com.wire.android.ui.home.conversations.multipartAttachmentsViewModel
 import com.wire.kalium.logic.data.asset.AssetTransferStatus
 import com.wire.kalium.logic.data.asset.isFailed
 import com.wire.kalium.logic.data.id.ConversationId
 import com.wire.kalium.logic.data.message.MessageAttachment
+import com.wire.android.feature.cells.R as cellsR
 
 /**
  * Displays a list of message attachments as a grid or a single attachment card.
@@ -66,17 +69,30 @@ fun MultipartAttachmentsView(
         else -> multipartAttachmentsViewModel(conversationId)
     }
 ) {
-    // Collect to trigger recomposition when offline availability changes.
-    val offlineAttachmentIds by viewModel.offlineAttachmentIds.collectAsStateWithLifecycle()
+    val context = LocalContext.current
+    val offlineAttachmentIds = viewModel.offlineAttachmentIds.collectAsStateWithLifecycle().value
+    val openLoadStates = viewModel.openLoadStates.collectAsStateWithLifecycle().value
+
+    LaunchedEffect(viewModel) {
+        viewModel.openAttachmentErrorEvent.collect {
+            Toast.makeText(context, cellsR.string.no_app_found, Toast.LENGTH_SHORT).show()
+        }
+    }
 
     // TODO I found out that empty attachments list is not handled here and it shows empty message with no information
     if (attachments.size == 1) {
         val attachment = attachments.first()
-        val item = remember(attachment, offlineAttachmentIds) {
-            viewModel.mapAttachment(attachment)
-        }
-        item.let {
-            AssetPreview(
+        val openLoadState = openLoadStates[attachment.assetId()]
+        val loadingProgress = (openLoadState as? MultipartAttachmentOpenLoadState.Loading)?.progress
+        val readyLocalPath = (openLoadState as? MultipartAttachmentOpenLoadState.Ready)?.localPath
+        val uiModel = attachment.toUiModel(
+            progress = loadingProgress,
+            isAvailableOffline = attachment.assetId() in offlineAttachmentIds,
+        ).copy(
+            openLoadState = openLoadState,
+            localPath = readyLocalPath ?: attachment.toUiModel().localPath,
+        )
+        AssetPreview(
                 modifier = modifier
                     .onVisibilityChanged { visible ->
                         if (visible) {
@@ -85,11 +101,11 @@ fun MultipartAttachmentsView(
                             viewModel.onAttachmentsHidden(attachments)
                         }
                     },
-                item = it,
+                item = uiModel,
                 messageStyle = messageStyle,
                 onClick = {
                     viewModel.onClick(
-                        attachment = it,
+                        attachment = uiModel,
                         openInImageViewer = onImageAttachmentClick,
                         openInVideoPlayer = { att ->
                             onVideoAttachmentClick(att.localPath, att.contentUrl, att.fileName)
@@ -100,11 +116,12 @@ fun MultipartAttachmentsView(
                     )
                 },
             )
-        }
     } else {
-        val groups = remember(attachments, offlineAttachmentIds) {
-            viewModel.mapAttachments(attachments = attachments)
-        }
+        val groups = viewModel.mapAttachments(
+            attachments = attachments,
+            offlineAttachmentIds = offlineAttachmentIds,
+            openLoadStates = openLoadStates,
+        )
 
         Column(
             modifier = modifier