TW-1702: handle download error on file tab

linagora · Jul 9, 2024 · b95f8f2 · b95f8f2
1 parent b9c3544
commit b95f8f2
Show file tree

Hide file tree

Showing 5 changed files with 28 additions and 6 deletions.
diff --git a/...ls/chat_details_page_view/files/chat_details_files_item/chat_details_files_item_view.dart b/...ls/chat_details_page_view/files/chat_details_files_item/chat_details_files_item_view.dart
@@ -24,7 +24,8 @@ class ChatDetailsFilesView extends StatelessWidget {
     return ValueListenableBuilder(
       valueListenable: controller.downloadFileStateNotifier,
       builder: (context, DownloadPresentationState state, child) {
-        if (state is DownloadingPresentationState) {
+        if (state is DownloadingPresentationState ||
+            state is DownloadErrorPresentationState) {
           return ChatDetailsDownloadingFileTile(
             mimeType: controller.event.mimeType,
             fileType: filetype,
@@ -37,6 +38,7 @@ class ChatDetailsFilesView extends StatelessWidget {
               controller.downloadManager
                   .cancelDownload(controller.event.eventId);
             },
+            hasError: state is DownloadErrorPresentationState,
           );
         } else if (state is DownloadedPresentationState) {
           return ChatDetailsDownloadedFileTile(

diff --git a/...details_page_view/files/chat_details_files_item_web/chat_details_files_item_view_web.dart b/...details_page_view/files/chat_details_files_item_web/chat_details_files_item_view_web.dart
@@ -22,7 +22,8 @@ class ChatDetailsFilesViewWeb extends StatelessWidget {
     return ValueListenableBuilder(
       valueListenable: controller.downloadFileStateNotifier,
       builder: (context, DownloadPresentationState state, child) {
-        if (state is DownloadingPresentationState) {
+        if (state is DownloadingPresentationState ||
+            state is DownloadErrorPresentationState) {
           return ChatDetailsFileTileRowDownloadingWeb(
             mimeType: controller.event.mimeType,
             fileType: filetype,
@@ -36,6 +37,7 @@ class ChatDetailsFilesViewWeb extends StatelessWidget {
               controller.downloadManager
                   .cancelDownload(controller.event.eventId);
             },
+            hasError: state is DownloadErrorPresentationState,
           );
         }
 

diff --git a/...at_details_page_view/files/chat_details_files_row/chat_details_file_downloading_tile.dart b/...at_details_page_view/files/chat_details_files_row/chat_details_file_downloading_tile.dart
@@ -14,6 +14,7 @@ class ChatDetailsDownloadingFileTile extends StatelessWidget {
     required this.fileType,
     required this.sizeString,
     required this.downloadFileStateNotifier,
+    this.hasError = false,
   });
 
   final GestureTapCallback onTap;
@@ -22,13 +23,15 @@ class ChatDetailsDownloadingFileTile extends StatelessWidget {
   final String? fileType;
   final String? sizeString;
   final ValueNotifier<DownloadPresentationState> downloadFileStateNotifier;
+  final bool hasError;
 
   @override
   Widget build(BuildContext context) {
     return InkWell(
       hoverColor: LinagoraSysColors.material().surfaceVariant,
       onTap: onTap,
       child: ChatDetailsFileRowDownloadingWrapper(
+        hasError: hasError,
         mimeType: mimeType,
         fileType: fileType,
         downloadFileStateNotifier: downloadFileStateNotifier,

diff --git a/...details_page_view/files/chat_details_files_row/chat_details_file_row_downloading_web.dart b/...details_page_view/files/chat_details_files_row/chat_details_file_row_downloading_web.dart
@@ -17,6 +17,7 @@ class ChatDetailsFileTileRowDownloadingWeb extends StatelessWidget {
     required this.sizeString,
     required this.onTap,
     required this.downloadFileStateNotifier,
+    this.hasError = false,
   });
 
   final GestureTapCallback onTap;
@@ -26,13 +27,15 @@ class ChatDetailsFileTileRowDownloadingWeb extends StatelessWidget {
   final String? fileType;
   final DateTime sentDate;
   final ValueNotifier<DownloadPresentationState> downloadFileStateNotifier;
+  final bool hasError;
 
   @override
   Widget build(BuildContext context) {
     return InkWell(
       hoverColor: LinagoraSysColors.material().surfaceVariant,
       onTap: onTap,
       child: ChatDetailsFileRowDownloadingWrapper(
+        hasError: hasError,
         mimeType: mimeType,
         fileType: fileType,
         downloadFileStateNotifier: downloadFileStateNotifier,

diff --git a/..._details_page_view/files/chat_details_files_row/chat_details_row_downloading_wrapper.dart b/..._details_page_view/files/chat_details_files_row/chat_details_row_downloading_wrapper.dart
@@ -9,13 +9,15 @@ class ChatDetailsFileRowDownloadingWrapper extends StatelessWidget {
   final String? mimeType;
   final String? fileType;
   final ValueNotifier<DownloadPresentationState> downloadFileStateNotifier;
+  final bool hasError;
 
   const ChatDetailsFileRowDownloadingWrapper({
     super.key,
     required this.child,
     required this.mimeType,
     required this.fileType,
     required this.downloadFileStateNotifier,
+    this.hasError = false,
   });
 
   final style = const MessageFileTileStyle();
@@ -50,11 +52,14 @@ class ChatDetailsFileRowDownloadingWrapper extends StatelessWidget {
                   width: style.iconSize,
                   height: style.iconSize,
                   decoration: BoxDecoration(
-                    color: Theme.of(context).colorScheme.primary,
+                    color: style.iconBackgroundColor(
+                      hasError: hasError,
+                      context: context,
+                    ),
                     shape: BoxShape.circle,
                   ),
                 ),
-                if (downloadProgress != 0)
+                if (downloadProgress != 0 && !hasError)
                   SizedBox(
                     width: style.circularProgressLoadingSize,
                     height: style.circularProgressLoadingSize,
@@ -66,11 +71,18 @@ class ChatDetailsFileRowDownloadingWrapper extends StatelessWidget {
                 Container(
                   width: style.downloadIconSize,
                   decoration: BoxDecoration(
-                    color: Theme.of(context).colorScheme.primary,
+                    color: style.iconBackgroundColor(
+                      hasError: hasError,
+                      context: context,
+                    ),
                     shape: BoxShape.circle,
                   ),
                   child: Icon(
-                    downloadProgress == 0 ? Icons.arrow_downward : Icons.close,
+                    hasError
+                        ? Icons.error_outline
+                        : downloadProgress == 0
+                            ? Icons.arrow_downward
+                            : Icons.close,
                     key: ValueKey(downloadProgress),
                     color: Theme.of(context).colorScheme.surface,
                     size: style.downloadIconSize,