Minor refactoring

lib/ui/create_collection_sheet.dart

@@ -54,17 +54,17 @@ String _actionName(CollectionActionType type, bool plural) {
   return addTitleSuffix ? text + titleSuffix : text;
 }
 
-void createCollectionSheet(
+void showCollectionActionSheet(
+  BuildContext context, {
   SelectedFiles? selectedFiles,
   List<SharedMediaFile>? sharedFiles,
-  BuildContext context, {
   CollectionActionType actionType = CollectionActionType.addFiles,
   bool showOptionToCreateNewAlbum = true,
 }) {
   showBarModalBottomSheet(
     context: context,
     builder: (context) {
-      return CreateCollectionSheet(
+      return CollectionActionSheet(
         selectedFiles: selectedFiles,
         sharedFiles: sharedFiles,
         actionType: actionType,
@@ -84,12 +84,12 @@ void createCollectionSheet(
   );
 }
 
-class CreateCollectionSheet extends StatefulWidget {
+class CollectionActionSheet extends StatefulWidget {
   final SelectedFiles? selectedFiles;
   final List<SharedMediaFile>? sharedFiles;
   final CollectionActionType actionType;
   final bool showOptionToCreateNewAlbum;
-  const CreateCollectionSheet({
+  const CollectionActionSheet({
     required this.selectedFiles,
     required this.sharedFiles,
     required this.actionType,
@@ -98,11 +98,11 @@ class CreateCollectionSheet extends StatefulWidget {
   });
 
   @override
-  State<CreateCollectionSheet> createState() => _CreateCollectionSheetState();
+  State<CollectionActionSheet> createState() => _CollectionActionSheetState();
 }
 
-class _CreateCollectionSheetState extends State<CreateCollectionSheet> {
-  final _logger = Logger((_CreateCollectionSheetState).toString());
+class _CollectionActionSheetState extends State<CollectionActionSheet> {
+  final _logger = Logger((_CollectionActionSheetState).toString());
 
   @override
   Widget build(BuildContext context) {

lib/ui/home_widget.dart

@@ -336,10 +336,9 @@ class _HomeWidgetState extends State<HomeWidget> {
       _shouldRenderCreateCollectionSheet = false;
       ReceiveSharingIntent.reset();
       Future.delayed(const Duration(milliseconds: 10), () {
-        createCollectionSheet(
-          null,
-          _sharedFiles,
+        showCollectionActionSheet(
           context,
+          sharedFiles: _sharedFiles,
           actionType: CollectionActionType.addFiles,
         );
       });

lib/ui/viewer/actions/file_selection_actions_widget.dart

@@ -283,10 +283,9 @@ class _FileSelectionActionWidgetState extends State<FileSelectionActionWidget> {
       widget.selectedFiles
           .unSelectAll(split.ownedByOtherUsers.toSet(), skipNotify: true);
     }
-    createCollectionSheet(
-      widget.selectedFiles,
-      null,
+    showCollectionActionSheet(
       context,
+      selectedFiles: widget.selectedFiles,
       actionType: CollectionActionType.moveFiles,
     );
   }
@@ -296,11 +295,7 @@ class _FileSelectionActionWidgetState extends State<FileSelectionActionWidget> {
       widget.selectedFiles
           .unSelectAll(split.ownedByOtherUsers.toSet(), skipNotify: true);
     }
-    createCollectionSheet(
-      widget.selectedFiles,
-      null,
-      context,
-    );
+    showCollectionActionSheet(context, selectedFiles: widget.selectedFiles);
   }
 
   Future<void> _onDeleteClick() async {
@@ -381,10 +376,9 @@ class _FileSelectionActionWidgetState extends State<FileSelectionActionWidget> {
       widget.selectedFiles
           .unSelectAll(split.ownedByOtherUsers.toSet(), skipNotify: true);
     }
-    createCollectionSheet(
-      widget.selectedFiles,
-      null,
+    showCollectionActionSheet(
       context,
+      selectedFiles: widget.selectedFiles,
       actionType: CollectionActionType.unHide,
     );
   }
@@ -459,10 +453,9 @@ class _FileSelectionActionWidgetState extends State<FileSelectionActionWidget> {
   }
 
   void _restore() {
-    createCollectionSheet(
-      widget.selectedFiles,
-      null,
+    showCollectionActionSheet(
       context,
+      selectedFiles: widget.selectedFiles,
       actionType: CollectionActionType.restoreFiles,
     );
   }

lib/ui/viewer/actions/file_selection_overlay_bar.dart

@@ -85,10 +85,9 @@ class _FileSelectionOverlayBarState extends State<FileSelectionOverlayBar> {
           iconButtonType: IconButtonType.primary,
           iconColor: iconColor,
           onTap: () {
-            createCollectionSheet(
-              widget.selectedFiles,
-              null,
+            showCollectionActionSheet(
               context,
+              selectedFiles: widget.selectedFiles,
               actionType: CollectionActionType.unHide,
             );
           },

lib/ui/viewer/file/fading_app_bar.dart

@@ -269,12 +269,11 @@ class FadingAppBarState extends State<FadingAppBar> {
   }
 
   Future<void> _handleUnHideRequest(BuildContext context) async {
-    final s = SelectedFiles();
-    s.files.add(widget.file);
-    createCollectionSheet(
-      s,
-      null,
+    final selectedFiles = SelectedFiles();
+    selectedFiles.files.add(widget.file);
+    showCollectionActionSheet(
       context,
+      selectedFiles: selectedFiles,
       actionType: CollectionActionType.unHide,
     );
   }

lib/ui/viewer/file/fading_bottom_bar.dart

@@ -237,10 +237,9 @@ class FadingBottomBarState extends State<FadingBottomBar> {
             onPressed: () {
               final selectedFiles = SelectedFiles();
               selectedFiles.toggleSelection(widget.file);
-              createCollectionSheet(
-                selectedFiles,
-                null,
+              showCollectionActionSheet(
                 context,
+                selectedFiles: selectedFiles,
                 actionType: CollectionActionType.restoreFiles,
               );
             },