TF-3047 Invoke saveToGallery method when click download icon on attac…

…hment in EmailView
linagora · Sep 30, 2024 · 76c22e4 · 76c22e4
1 parent 013eaec
commit 76c22e4
Show file tree

Hide file tree

Showing 5 changed files with 205 additions and 99 deletions.
diff --git a/core/lib/presentation/views/dialog/downloading_file_dialog_builder.dart b/core/lib/presentation/views/dialog/downloading_file_dialog_builder.dart
@@ -1,7 +1,6 @@
 
-import 'package:core/core.dart';
+import 'package:core/presentation/extensions/color_extension.dart';
 import 'package:flutter/cupertino.dart';
-import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 
 typedef OnCancelDownloadActionClick = void Function();
@@ -36,36 +35,78 @@ class DownloadingFileDialogBuilder {
   }
 
   Widget build() {
-    return CupertinoAlertDialog(
+    return Dialog(
       key: _key ?? const Key('DownloadingFileBuilder'),
-      title: Text(_title, style: const TextStyle(fontSize: 17.0, color: Colors.black)),
-      content: Padding(
-        padding: const EdgeInsets.only(top: 16.0, left: 16.0, right: 16.0),
-        child: Center(
-          child: Column(
-            children: [
-              const SizedBox(
-                width: 20.0,
-                height: 20.0,
-                child: CupertinoActivityIndicator()),
-              const SizedBox(height: 16),
-              Text(
+      backgroundColor: Colors.white,
+      shape: const RoundedRectangleBorder(
+        borderRadius: BorderRadius.all(Radius.circular(12))
+      ),
+      alignment: Alignment.center,
+      child: SizedBox(
+        width: 250,
+        child: Column(
+          mainAxisSize: MainAxisSize.min,
+          mainAxisAlignment: MainAxisAlignment.center,
+          children: [
+            Padding(
+              padding: const EdgeInsetsDirectional.symmetric(
+                horizontal: 12,
+                vertical: 20
+              ),
+              child: Text(
+                _title,
+                style: const TextStyle(
+                  fontSize: 17.0,
+                  color: Colors.black,
+                  fontWeight: FontWeight.bold
+                )
+              ),
+            ),
+            const SizedBox(
+              width: 20.0,
+              height: 20.0,
+              child: CupertinoActivityIndicator()
+            ),
+            const SizedBox(height: 16),
+            Padding(
+              padding: const EdgeInsets.symmetric(
+                horizontal: 16,
+              ),
+              child: Text(
                 _content,
-                style: const TextStyle(fontSize: 13.0, color: Colors.black),
-                softWrap: false,
-                maxLines: 1)
-            ],
-          ),
-        )),
-      actions: [
-        if (_actionText.isNotEmpty)
-          Padding(
-            padding: const EdgeInsets.only(bottom: kIsWeb ? 16 : 0, top: kIsWeb ? 16 : 0),
-            child: TextButton(
-              onPressed: () => _onCancelDownloadActionClick?.call(),
-              child: Text(_actionText, style: const TextStyle(fontSize: 17.0, color: AppColor.appColor)),
-            ))
-      ],
+                style: const TextStyle(
+                  fontSize: 13.0,
+                  color: Colors.black,
+                ),
+                textAlign: TextAlign.center,
+                overflow: TextOverflow.ellipsis,
+                maxLines: 2
+              ),
+            ),
+            const SizedBox(height: 16),
+            if (_actionText.isNotEmpty && _onCancelDownloadActionClick != null)
+              ... [
+                const Divider(),
+                TextButton(
+                  onPressed: _onCancelDownloadActionClick,
+                  style: TextButton.styleFrom(
+                    backgroundColor: Colors.white,
+                    foregroundColor: AppColor.primaryColor,
+                    overlayColor: Colors.black12,
+                    fixedSize: const Size.fromWidth(250),
+                  ),
+                  child: Text(
+                    _actionText,
+                    style: const TextStyle(
+                      fontSize: 17.0,
+                      color: AppColor.primaryColor
+                    )
+                  ),
+                )
+              ]
+          ],
+        ),
+      ),
     );
   }
 }
diff --git a/ios/Runner/Info.plist b/ios/Runner/Info.plist
@@ -55,7 +55,7 @@
 		<key>NSContactsUsageDescription</key>
 		<string>Used by Team-Mail for Email address auto-completion. You can easily to find the email address in your contact book to send the email.</string>
 		<key>NSPhotoLibraryAddUsageDescription</key>
-		<string>To save photo to library, we need permission to access your photo library.</string>
+		<string>To save photos or videos to library, we need permission to access your photo library.</string>
 		<key>NSPhotoLibraryUsageDescription</key>
 		<string>To select and upload photo, we need permission to access your photo library.</string>
 		<key>UIBackgroundModes</key>

diff --git a/lib/features/base/mixin/save_media_to_gallery_mixin.dart b/lib/features/base/mixin/save_media_to_gallery_mixin.dart
@@ -15,7 +15,7 @@ import 'package:tmail_ui_user/main/permissions/storage_permission_service.dart';
 typedef OnSaveCallbackAction = Function(bool isSuccess);
 
 mixin SaveMediaToGalleryMixin {
-  Future<void> handleAndroidStoragePermission(BuildContext context) async {
+  Future<void> _handleAndroidStoragePermission(BuildContext context) async {
     if (await StoragePermissionService().isUserHaveToRequestStoragePermissionAndroid()) {
       final permission = await Permission.storage.request();
 
@@ -32,8 +32,7 @@ mixin SaveMediaToGalleryMixin {
                   AppLocalizations.of(context).app_name,
                 ),
               ),
-              onAcceptButton: () =>
-                  StoragePermissionService().goToSettingsForPermissionActions(),
+              onAcceptButton: StoragePermissionService().goToSettingsForPermissionActions,
             );
           },
         );
@@ -46,7 +45,7 @@ mixin SaveMediaToGalleryMixin {
     }
   }
 
-  Future<void> handlePhotoPermissionIOS(BuildContext context) async {
+  Future<void> _handlePhotoPermissionIOS(BuildContext context) async {
     final permissionStatus = await StoragePermissionService().requestPhotoAddOnlyPermissionIOS();
     if (permissionStatus.isPermanentlyDenied && context.mounted) {
       showDialog(
@@ -61,8 +60,7 @@ mixin SaveMediaToGalleryMixin {
                 AppLocalizations.of(context).app_name,
               ),
             ),
-            onAcceptButton: () =>
-                StoragePermissionService().goToSettingsForPermissionActions(),
+            onAcceptButton: StoragePermissionService().goToSettingsForPermissionActions,
           );
         },
       );
@@ -72,36 +70,35 @@ mixin SaveMediaToGalleryMixin {
     }
   }
 
-  Future<void> saveMediaToGallery({
+  Future<void> _saveMediaToGallery({
     required File fileInDownloadsInApp,
     required MediaType mediaType,
     OnSaveCallbackAction? onSaveCallbackAction
   }) async {
     if (mediaType.isImageFile()) {
-      await saveImageToGallery(file: fileInDownloadsInApp);
+      await _saveImageToGallery(file: fileInDownloadsInApp);
     } else if (mediaType.isVideoFile()) {
-      await saveVideoToGallery(file: fileInDownloadsInApp);
+      await _saveVideoToGallery(file: fileInDownloadsInApp);
     } else {
       return;
     }
     onSaveCallbackAction?.call(true);
   }
 
-  Future<void> saveImageToGallery({
+  Future<void> _saveImageToGallery({
     required File file,
   }) async {
    log('SaveMediaToGalleryMixin::saveImageToGallery:file path: ${file.path}');
     await Gal.putImage(file.path);
   }
 
-  Future<void> saveVideoToGallery({
+  Future<void> _saveVideoToGallery({
     required File file,
   }) async {
     log('SaveMediaToGalleryMixin::saveVideoToGallery:file path: ${file.path}');
     await Gal.putVideo(file.path);
   }
 
-
   Future<void> saveToGallery({
     required BuildContext context,
     required String filePath,
@@ -110,17 +107,17 @@ mixin SaveMediaToGalleryMixin {
   }) async {
     try {
       if (PlatformInfo.isAndroid) {
-        await handleAndroidStoragePermission(context);
+        await _handleAndroidStoragePermission(context);
       } else if (PlatformInfo.isIOS) {
-        await handlePhotoPermissionIOS(context);
+        await _handlePhotoPermissionIOS(context);
       } else {
         return;
       }
 
       final fileInDownloadsInApp = File(filePath);
 
       if (context.mounted) {
-        await saveMediaToGallery(
+        await _saveMediaToGallery(
           mediaType: mediaType,
           fileInDownloadsInApp: fileInDownloadsInApp,
           onSaveCallbackAction: onSaveCallbackAction