fix: modify download placeholders to clarify saved and downloaded cas…

…es [WPB-4732] (#2322)
wireapp · Oct 12, 2023 · 052384e · 052384e
1 parent e096a5b
commit 052384e
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 17 deletions.
diff --git a/...tlin/com/wire/android/ui/home/conversations/model/messagetypes/asset/AssetMessageTypes.kt b/...tlin/com/wire/android/ui/home/conversations/model/messagetypes/asset/AssetMessageTypes.kt
@@ -39,7 +39,6 @@ import androidx.compose.foundation.layout.width
 import androidx.compose.foundation.layout.wrapContentWidth
 import androidx.compose.foundation.shape.RoundedCornerShape
 import androidx.compose.material3.Card
-import androidx.compose.material3.Icon
 import androidx.compose.material3.MaterialTheme
 import androidx.compose.material3.Text
 import androidx.compose.runtime.Composable
@@ -301,21 +300,6 @@ private fun DownloadStatusIcon(assetDownloadStatus: Message.DownloadStatus, asse
             size = dimensions().spacing16x
         )
 
-        assetUploadStatus == Message.UploadStatus.FAILED_UPLOAD -> {}
-        assetDownloadStatus == Message.DownloadStatus.SAVED_INTERNALLY -> Icon(
-            painter = painterResource(id = R.drawable.ic_download),
-            contentDescription = stringResource(R.string.content_description_download_icon),
-            modifier = Modifier.size(dimensions().wireIconButtonSize),
-            tint = MaterialTheme.wireColorScheme.secondaryText
-        )
-
-        assetDownloadStatus == Message.DownloadStatus.SAVED_EXTERNALLY -> Icon(
-            painter = painterResource(id = R.drawable.ic_check_tick),
-            contentDescription = stringResource(R.string.content_description_check),
-            modifier = Modifier.size(dimensions().wireIconButtonSize),
-            tint = MaterialTheme.wireColorScheme.secondaryText
-        )
-
         else -> {}
     }
 }

diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
@@ -405,7 +405,7 @@
     <string name="asset_message_tap_to_download_text">Tap to download</string>
     <string name="asset_message_upload_in_progress_text">Uploading…</string>
     <string name="asset_message_download_in_progress_text">Downloading…</string>
-    <string name="asset_message_downloaded_internally_text">Downloaded</string>
+    <string name="asset_message_downloaded_internally_text">Tap to view</string>
     <string name="asset_message_saved_externally_text">Saved</string>
     <string name="asset_message_failed_download_text">File not available</string>
     <string name="asset_message_failed_upload_text">File upload failed</string>