chore: defaultly save files on Android

- defaultly save files on Android instead of share
- add dedicated share button for files and images on Android
- use ListTile instead of row to display file event
- update file_picker_cross

Signed-off-by: TheOneWithTheBraid <the-one@with-the-braid.cf>

lib/pages/chat/events/message_download_content.dart

@@ -1,8 +1,10 @@
 import 'package:flutter/material.dart';
 
+import 'package:flutter_gen/gen_l10n/l10n.dart';
 import 'package:matrix/matrix.dart';
 
 import 'package:fluffychat/utils/matrix_sdk_extensions.dart/event_extension.dart';
+import 'package:fluffychat/utils/platform_infos.dart';
 
 class MessageDownloadContent extends StatelessWidget {
   final Event event;
@@ -28,24 +30,26 @@ class MessageDownloadContent extends StatelessWidget {
         crossAxisAlignment: CrossAxisAlignment.start,
         mainAxisSize: MainAxisSize.min,
         children: <Widget>[
-          Row(
-            children: [
-              Icon(
-                Icons.file_download_outlined,
+          ListTile(
+            leading: Icon(
+              Icons.file_download_outlined,
+              color: textColor,
+            ),
+            title: Text(
+              filename,
+              maxLines: 1,
+              style: TextStyle(
                 color: textColor,
+                fontWeight: FontWeight.bold,
               ),
-              const SizedBox(width: 8),
-              Expanded(
-                child: Text(
-                  filename,
-                  maxLines: 1,
-                  style: TextStyle(
-                    color: textColor,
-                    fontWeight: FontWeight.bold,
-                  ),
-                ),
-              ),
-            ],
+            ),
+            trailing: PlatformInfos.isAndroid
+                ? IconButton(
+                    onPressed: () => event.shareFile(context),
+                    tooltip: L10n.of(context)!.share,
+                    icon: const Icon(Icons.share),
+                  )
+                : null,
           ),
           const Divider(),
           Row(

lib/pages/image_viewer/image_viewer.dart

@@ -28,6 +28,9 @@ class ImageViewerController extends State<ImageViewer> {
   /// Save this file with a system call.
   void saveFileAction() => widget.event.saveFile(context);
 
+  /// Save this file with a system call.
+  void shareFileAction() => widget.event.shareFile(context);
+
   static const maxScaleFactor = 1.5;
 
   /// Go back if user swiped it away

lib/pages/image_viewer/image_viewer_view.dart

@@ -3,6 +3,7 @@ import 'package:flutter/material.dart';
 import 'package:flutter_gen/gen_l10n/l10n.dart';
 
 import 'package:fluffychat/pages/chat/events/image_bubble.dart';
+import 'package:fluffychat/utils/platform_infos.dart';
 import 'image_viewer.dart';
 
 class ImageViewerView extends StatelessWidget {
@@ -37,6 +38,13 @@ class ImageViewerView extends StatelessWidget {
             color: Colors.white,
             tooltip: L10n.of(context)!.downloadFile,
           ),
+          if (PlatformInfos.isAndroid)
+            IconButton(
+              onPressed: controller.shareFileAction,
+              tooltip: L10n.of(context)!.share,
+              color: Colors.white,
+              icon: const Icon(Icons.share),
+            )
         ],
       ),
       body: InteractiveViewer(

lib/utils/matrix_sdk_extensions.dart/event_extension.dart

@@ -8,18 +8,28 @@ import 'package:matrix/matrix.dart';
 import 'matrix_file_extension.dart';
 
 extension LocalizedBody on Event {
+  Future<LoadingDialogResult<MatrixFile?>> _getFile(BuildContext context) =>
+      showFutureLoadingDialog(
+        context: context,
+        future: () => downloadAndDecryptAttachmentCached(),
+      );
+
   void saveFile(BuildContext context) async {
-    final matrixFile = await showFutureLoadingDialog(
-      context: context,
-      future: () => downloadAndDecryptAttachmentCached(),
-    );
+    final matrixFile = await _getFile(context);
 
     matrixFile.result?.save(context);
   }
 
+  void shareFile(BuildContext context) async {
+    final matrixFile = await _getFile(context);
+
+    matrixFile.result?.share(context);
+  }
+
   bool get isAttachmentSmallEnough =>
       infoMap['size'] is int &&
       infoMap['size'] < room.client.database!.maxFileSize;
+
   bool get isThumbnailSmallEnough =>
       thumbnailInfoMap['size'] is int &&
       thumbnailInfoMap['size'] < room.client.database!.maxFileSize;

lib/utils/matrix_sdk_extensions.dart/matrix_file_extension.dart

@@ -13,23 +13,25 @@ import 'package:fluffychat/utils/platform_infos.dart';
 extension MatrixFileExtension on MatrixFile {
   void save(BuildContext context) async {
     final fileName = name.split('/').last;
-    if (PlatformInfos.isMobile) {
-      final tmpDirectory = PlatformInfos.isAndroid
-          ? (await getExternalStorageDirectories(
-                  type: StorageDirectory.downloads))!
-              .first
-          : await getTemporaryDirectory();
-      final path = '${tmpDirectory.path}$fileName';
-      await File(path).writeAsBytes(bytes);
-      await Share.shareFiles([path]);
-      ScaffoldMessenger.of(context).showSnackBar(
-        SnackBar(content: Text(L10n.of(context)!.savedFileAs(path))),
-      );
-      return;
-    } else {
-      final file = FilePickerCross(bytes);
-      await file.exportToStorage(fileName: fileName);
-    }
+
+    final file = FilePickerCross(bytes);
+    await file.exportToStorage(fileName: fileName, share: false);
+  }
+
+  void share(BuildContext context) async {
+    final fileName = name.split('/').last;
+    final tmpDirectory = PlatformInfos.isAndroid
+        ? (await getExternalStorageDirectories(
+                type: StorageDirectory.downloads))!
+            .first
+        : await getTemporaryDirectory();
+    final path = '${tmpDirectory.path}$fileName';
+    await File(path).writeAsBytes(bytes);
+    await Share.shareFiles([path]);
+    ScaffoldMessenger.of(context).showSnackBar(
+      SnackBar(content: Text(L10n.of(context)!.savedFileAs(path))),
+    );
+    return;
   }
 
   MatrixFile get detectFileType {