Merge pull request #998 from nextcloud/refactor/neon/dialogs

Refactor/neon/dialogs

packages/neon/neon_files/lib/l10n/en.arb

@@ -1,7 +1,5 @@
 {
   "@@locale": "en",
-  "actionYes": "Yes",
-  "actionNo": "No",
   "actionDelete": "Delete",
   "actionRename": "Rename",
   "actionMove": "Move",
@@ -43,6 +41,8 @@
       }
     }
   },
+  "actionDeleteTitle": "Permanently delete?",
+  "filesChooseCreate": "Add to Nextcloud",
   "folderCreate": "Create folder",
   "folderName": "Folder name",
   "folderRename": "Rename folder",

packages/neon/neon_files/lib/l10n/localizations.dart

@@ -89,18 +89,6 @@ abstract class FilesLocalizations {
   /// A list of this localizations delegate's supported locales.
   static const List<Locale> supportedLocales = <Locale>[Locale('en')];
 
-  /// No description provided for @actionYes.
-  ///
-  /// In en, this message translates to:
-  /// **'Yes'**
-  String get actionYes;
-
-  /// No description provided for @actionNo.
-  ///
-  /// In en, this message translates to:
-  /// **'No'**
-  String get actionNo;
-
   /// No description provided for @actionDelete.
   ///
   /// In en, this message translates to:
@@ -185,6 +173,18 @@ abstract class FilesLocalizations {
   /// **'Are you sure you want to download a file that is bigger than {warningSize} ({actualSize})?'**
   String downloadConfirmSizeWarning(String warningSize, String actualSize);
 
+  /// No description provided for @actionDeleteTitle.
+  ///
+  /// In en, this message translates to:
+  /// **'Permanently delete?'**
+  String get actionDeleteTitle;
+
+  /// No description provided for @filesChooseCreate.
+  ///
+  /// In en, this message translates to:
+  /// **'Add to Nextcloud'**
+  String get filesChooseCreate;
+
   /// No description provided for @folderCreate.
   ///
   /// In en, this message translates to:

packages/neon/neon_files/lib/l10n/localizations_en.dart

@@ -4,12 +4,6 @@ import 'localizations.dart';
 class FilesLocalizationsEn extends FilesLocalizations {
   FilesLocalizationsEn([String locale = 'en']) : super(locale);
 
-  @override
-  String get actionYes => 'Yes';
-
-  @override
-  String get actionNo => 'No';
-
   @override
   String get actionDelete => 'Delete';
 
@@ -58,6 +52,12 @@ class FilesLocalizationsEn extends FilesLocalizations {
     return 'Are you sure you want to download a file that is bigger than $warningSize ($actualSize)?';
   }
 
+  @override
+  String get actionDeleteTitle => 'Permanently delete?';
+
+  @override
+  String get filesChooseCreate => 'Add to Nextcloud';
+
   @override
   String get folderCreate => 'Create folder';
 

packages/neon/neon_files/lib/src/blocs/browser.dart

@@ -8,17 +8,33 @@ import 'package:neon_framework/utils.dart';
 import 'package:nextcloud/webdav.dart';
 import 'package:rxdart/rxdart.dart';
 
+/// Mode to operate the `FilesBrowserView` in.
+enum FilesBrowserMode {
+  /// Default file browser mode.
+  ///
+  /// When a file is selected it will be opened or downloaded.
+  browser,
+
+  /// Select directory.
+  selectDirectory,
+
+  /// Do not show file actions.
+  noActions,
+}
+
 sealed class FilesBrowserBloc implements InteractiveBloc {
   @internal
   factory FilesBrowserBloc(
     final FilesOptions options,
     final Account account, {
     final PathUri? initialPath,
+    final FilesBrowserMode? mode,
   }) =>
       _FilesBrowserBloc(
         options,
         account,
         initialPath: initialPath,
+        mode: mode,
       );
 
   void setPath(final PathUri uri);
@@ -30,27 +46,41 @@ sealed class FilesBrowserBloc implements InteractiveBloc {
   BehaviorSubject<PathUri> get uri;
 
   FilesOptions get options;
+
+  /// Mode to operate the `FilesBrowserView` in.
+  FilesBrowserMode get mode;
 }
 
 class _FilesBrowserBloc extends InteractiveBloc implements FilesBrowserBloc {
   _FilesBrowserBloc(
     this.options,
     this.account, {
-    final PathUri? initialPath,
-  }) {
-    if (initialPath != null) {
-      uri.add(initialPath);
+    this.initialPath,
+    final FilesBrowserMode? mode,
+  }) : mode = mode ?? FilesBrowserMode.browser {
+    final parent = initialPath?.parent;
+    if (parent != null) {
+      uri.add(parent);
     }
 
+    options.showHiddenFilesOption.addListener(refresh);
+
     unawaited(refresh());
   }
 
   @override
   final FilesOptions options;
   final Account account;
 
+  @override
+  final FilesBrowserMode mode;
+
+  final PathUri? initialPath;
+
   @override
   void dispose() {
+    options.showHiddenFilesOption.removeListener(refresh);
+
     unawaited(files.close());
     unawaited(uri.close());
     super.dispose();
@@ -80,7 +110,28 @@ class _FilesBrowserBloc extends InteractiveBloc implements FilesBrowserBloc {
         ),
         depth: WebDavDepth.one,
       ),
-      (final response) => response.toWebDavFiles().sublist(1),
+      (final response) {
+        final unwrapped = response.toWebDavFiles().sublist(1);
+
+        return unwrapped.where((final file) {
+          // Do not show files when selecting a directory
+          if (mode == FilesBrowserMode.selectDirectory && !file.isDirectory) {
+            return false;
+          }
+
+          // Do not show itself when selecting a directory
+          if (mode == FilesBrowserMode.selectDirectory && initialPath == file.path) {
+            return false;
+          }
+
+          // Do not show hidden files unless the option is enabled
+          if (!options.showHiddenFilesOption.value && file.isHidden) {
+            return false;
+          }
+
+          return true;
+        }).toList();
+      },
       emitEmptyCache: true,
     );
   }

packages/neon/neon_files/lib/src/blocs/files.dart

@@ -56,7 +56,7 @@ sealed class FilesBloc implements InteractiveBloc {
 
   FilesBrowserBloc get browser;
 
-  FilesBrowserBloc getNewFilesBrowserBloc({final PathUri? initialUri});
+  FilesBrowserBloc getNewFilesBrowserBloc({final PathUri? initialUri, final FilesBrowserMode? mode});
 }
 
 class _FilesBloc extends InteractiveBloc implements FilesBloc {
@@ -235,13 +235,12 @@ class _FilesBloc extends InteractiveBloc implements FilesBloc {
   }
 
   @override
-  FilesBrowserBloc getNewFilesBrowserBloc({
-    final PathUri? initialUri,
-  }) =>
+  FilesBrowserBloc getNewFilesBrowserBloc({final PathUri? initialUri, final FilesBrowserMode? mode}) =>
       FilesBrowserBloc(
         options,
         account,
         initialPath: initialUri,
+        mode: mode,
       );
 
   void downloadParallelismListener() {