added action bar for selected bookmark item

lib/main.dart

@@ -7,7 +7,6 @@ import 'pages/collections_list_page.dart';
 import 'pages/search_page.dart';
 import 'pages/settings_page.dart';
 import 'service/search_provider.dart';
-import 'service/settings_provider.dart';
 import 'service/shared_link_provider.dart';
 import 'service/storage.dart';
 import 'service/share_intent_service.dart';
@@ -21,7 +20,6 @@ void main() async {
       providers: [
         ChangeNotifierProvider(create: (_) => SharedLinkProvider()),
         ChangeNotifierProvider(create: (_) => SearchProvider()),
-        ChangeNotifierProvider(create: (_) => SettingsProvider()),
       ],
       child: const MapsBookmarks(),
     ),

lib/model/settings.dart

@@ -1,39 +0,0 @@
-import '../assets/constants.dart' as constants;
-
-class Settings {
-  final String exportDirectoryPath;
-  final bool alwaysExportEnabled;
-
-  Settings._({
-    required this.exportDirectoryPath,
-    required this.alwaysExportEnabled,
-  });
-
-  Map<String, dynamic> toJson() {
-    return {
-      'exportDirectoryPath': exportDirectoryPath,
-      'alwaysExportEnabled': alwaysExportEnabled,
-    };
-  }
-
-  factory Settings.fromJson(Map<String, dynamic> json) {
-    return Settings._(
-      exportDirectoryPath: json['exportDirectoryPath'] as String,
-      alwaysExportEnabled: json['alwaysExportEnabled'] as bool,
-    );
-  }
-
-  factory Settings.defaults() {
-    return Settings._(
-      exportDirectoryPath: constants.defaultAndroidExportDirectory,
-      alwaysExportEnabled: false,
-    );
-  }
-
-  Settings copyWith({String? exportDirectoryPath, bool? alwaysExportEnabled}) {
-    return Settings._(
-      exportDirectoryPath: exportDirectoryPath ?? this.exportDirectoryPath,
-      alwaysExportEnabled: alwaysExportEnabled ?? this.alwaysExportEnabled,
-    );
-  }
-}

lib/pages/collection_page.dart

@@ -9,6 +9,7 @@ import '../service/notifying.dart';
 import '../service/shared_link_provider.dart';
 import '../service/storage.dart';
 import '../service/url_launcher.dart';
+import '../widgets/collection_page_widgets/list_item_actions_widget.dart';
 import '../widgets/create_bookmark_dialog.dart';
 
 class CollectionPage extends StatefulWidget {
@@ -22,6 +23,7 @@ class CollectionPage extends StatefulWidget {
 
 class _CollectionPageState extends State<CollectionPage> {
   MapsLinkMetadata? selectedMapsLink;
+  int selectedBookmarkId = -1;
 
   @override
   void initState() {
@@ -52,29 +54,46 @@ class _CollectionPageState extends State<CollectionPage> {
         ).whenComplete(() => setState(() {}));
       },
     ),
-  );
+  ).whenComplete(deselectBookmark);
 
   void onBookmarkSaved(Bookmark bookmark) {
     Storage.addOrUpdateBookmark(bookmark);
     setState(() {});
-    context.read<SharedLinkProvider>().removeCurrentMapsLink();
+    Provider.of<SharedLinkProvider>(
+      context,
+      listen: false,
+    ).removeCurrentMapsLink();
   }
 
   Widget bookmarksListItemBuilder(BuildContext context, Bookmark bookmark) {
+    final selected = selectedBookmarkId == bookmark.id;
     return ListTile(
       title: Text(bookmark.name),
-      onTap: () => launchUrlFromString(bookmark.link).then((errorCode) {
+      selected: selected,
-        if (context.mounted) {
+      onTap: () {
-          return Notifying.showUrlErrorSnackbar(context, errorCode);
+        if (selected) {
+          onCancelSelectionPressed();
+          setState(() {});
+        } else if (selectedBookmarkId != -1 && !selected) {
+          selectedBookmarkId = bookmark.id;
+          setState(() {});
+        } else {
+          launchUrlFromString(bookmark.link).then((errorCode) {
+            if (context.mounted) {
+              return Notifying.showUrlErrorSnackbar(context, errorCode);
+            }
+          });
         }
+      },
+      onLongPress: () => setState(() {
+        selectedBookmarkId = bookmark.id;
       }),
-      onLongPress: () => editBookmark(bookmark),
     );
   }
 
   @override
   Widget build(BuildContext context) {
-    SharedLinkProvider provider = context.watch<SharedLinkProvider>();
+    SharedLinkProvider provider = Provider.of<SharedLinkProvider>(context);
     selectedMapsLink = provider.currentMapsLinkMetadata;
 
     if (BookmarksProvider.selectedCollectionId == null) {
@@ -87,36 +106,69 @@ class _CollectionPageState extends State<CollectionPage> {
       (c) => c.id == BookmarksProvider.selectedCollectionId,
     );
 
-    return Scaffold(
+    return PopScope(
-      appBar: AppBar(
+      canPop: selectedBookmarkId == -1,
-        title: selectedMapsLink != null
+      onPopInvokedWithResult: (didPop, result) {
-            ? Text(
+        if (didPop == false) deselectBookmark();
-                AppLocalizations.of(context)!.addToCollection(collection.name),
+      },
+      child: Scaffold(
+        appBar: AppBar(
+          title: selectedMapsLink != null
+              ? Text(
+                  AppLocalizations.of(
+                    context,
+                  )!.addToCollection(collection.name),
+                )
+              : Text(collection.name),
+          actions: [
+            if (selectedMapsLink != null)
+              TextButton(
+                onPressed: () => provider.removeCurrentMapsLink(),
+                child: Text(AppLocalizations.of(context)!.cancel),
+              ),
+          ],
+        ),
+        bottomNavigationBar: selectedBookmarkId > 0
+            ? ListItemActionsWidget(
+                onDeletePressed: onDeleteBookmarkPressed,
+                onCancelPressed: onCancelSelectionPressed,
+                onEditPressed: () => editBookmark(
+                  bookmarks.firstWhere(
+                    (element) => element.id == selectedBookmarkId,
+                  ),
+                ),
               )
-            : Text(collection.name),
+            : null,
-        actions: [
+        body: Center(
-          if (selectedMapsLink != null)
+          child: SizedBox(
-            TextButton(
+            width: MediaQuery.of(context).size.width * 0.9,
-              onPressed: () => provider.removeCurrentMapsLink(),
+            child: ListView.separated(
-              child: Text(AppLocalizations.of(context)!.cancel),
+              itemBuilder: (context, index) =>
+                  bookmarksListItemBuilder(context, bookmarks.elementAt(index)),
+              itemCount: bookmarks.length,
+              separatorBuilder: (context, index) => SizedBox(height: 10),
             ),
-        ],
-      ),
-      body: Center(
-        child: SizedBox(
-          width: MediaQuery.of(context).size.width * 0.9,
-          child: ListView.separated(
-            itemBuilder: (context, index) =>
-                bookmarksListItemBuilder(context, bookmarks.elementAt(index)),
-            itemCount: bookmarks.length,
-            separatorBuilder: (context, index) => SizedBox(height: 10),
           ),
         ),
-      ),
+        floatingActionButton: FloatingActionButton(
-      floatingActionButton: FloatingActionButton(
+          onPressed: onAddButtonPressed,
-        onPressed: onAddButtonPressed,
+          child: Icon(selectedMapsLink != null ? Icons.save : Icons.add),
-        child: Icon(selectedMapsLink != null ? Icons.save : Icons.add),
+        ),
       ),
     );
   }
+
+  void onCancelSelectionPressed() => deselectBookmark();
+
+  void onDeleteBookmarkPressed() {
+    Storage.deleteBookmarkById(
+      selectedBookmarkId,
+    ).whenComplete(() => setState(() {}));
+    deselectBookmark();
+  }
+
+  void deselectBookmark() {
+    selectedBookmarkId = -1;
+    setState(() {});
+  }
 }

lib/pages/collections_list_page.dart

@@ -87,7 +87,8 @@ class _CollectionsListPageState extends State<CollectionsListPage> {
     final collections = Storage.loadCollections();
     bookmarkCountMap = Storage.loadPerCollectionBookmarkCount();
     addingNewBookmark =
-        context.watch<SharedLinkProvider>().currentMapsLinkMetadata != null;
+        Provider.of<SharedLinkProvider>(context).currentMapsLinkMetadata !=
+        null;
     return Scaffold(
       appBar: AppBar(
         title: addingNewBookmark
@@ -96,8 +97,10 @@ class _CollectionsListPageState extends State<CollectionsListPage> {
         actions: [
           if (addingNewBookmark)
             TextButton(
-              onPressed: () =>
+              onPressed: () => Provider.of<SharedLinkProvider>(
-                  context.read<SharedLinkProvider>().removeCurrentMapsLink(),
+                context,
+                listen: false,
+              ).removeCurrentMapsLink(),
               child: Text(AppLocalizations.of(context)!.cancel),
             )
           else

lib/pages/settings_page.dart

@@ -2,12 +2,10 @@ import 'dart:async';
 
 import 'package:flutter/material.dart';
 import 'package:permission_handler/permission_handler.dart';
-import 'package:provider/provider.dart';
 
 import '../l10n/app_localizations.dart';
 import '../service/notifying.dart';
 import '../service/permission_service.dart';
-import '../service/settings_provider.dart';
 import '../service/storage.dart';
 
 class SettingsPage extends StatefulWidget {
@@ -31,16 +29,10 @@ class _SettingsPageState extends State<SettingsPage> {
     super.initState();
   }
 
-  // TODO: Localize
   @override
   Widget build(BuildContext context) {
     final titlePadding = Theme.of(context).listTileTheme.contentPadding!;
     checkStoragePermission();
-    final alwaysExportEnabled = context
-        .watch<SettingsProvider>()
-        .settings
-        .alwaysExportEnabled;
-
     return Scaffold(
       appBar: AppBar(title: Text(AppLocalizations.of(context)!.settings)),
       body: Center(
@@ -59,11 +51,9 @@ class _SettingsPageState extends State<SettingsPage> {
               SizedBox(height: tileSpacing),
               ListTile(
                 title: Text('Grant storage permisson'),
-                subtitle: storagePermissionIsGranted
+                subtitle: Text(
-                    ? Text('Storage permission granted')
+                  'For app-data settings to work, you need to grant the app permissions to manage internal storage.',
-                    : Text(
+                ),
-                        'For app-data settings to work, you need to grant the app permissions to manage internal storage.',
-                      ),
                 onTap: () => PermissionService.requestStoragePermission
                     .whenComplete(() => checkStoragePermission()),
                 trailing: Icon(Icons.arrow_forward_ios_rounded),
@@ -73,7 +63,7 @@ class _SettingsPageState extends State<SettingsPage> {
               ListTile(
                 title: Text(AppLocalizations.of(context)!.import),
                 subtitle: Text('Import app-data from a json file.'),
-                onTap: () => onJsonImportPressed(),
+                onTap: () => onActivateJsonImportPressed(),
                 trailing: Icon(Icons.arrow_forward_ios_rounded),
                 enabled: storagePermissionIsGranted,
               ),
@@ -83,39 +73,10 @@ class _SettingsPageState extends State<SettingsPage> {
                 subtitle: Text(
                   'Export app-data to a json file in the selected directory.',
                 ),
-                onTap: () => onJsonExportPressed(),
+                onTap: () => onActivateJsonExportPressed(),
                 trailing: Icon(Icons.arrow_forward_ios_rounded),
                 enabled: storagePermissionIsGranted,
               ),
-              SizedBox(height: tileSpacing),
-              ListTile(
-                title: Text('Always save to file'),
-                subtitle: Text(
-                  'Export app data to a directory, every time you make a change',
-                ),
-                onTap: () => onAlwaysSaveToJsonPressed(),
-                trailing: Checkbox(
-                  value: alwaysExportEnabled,
-                  onChanged: (value) {
-                    onAlwaysSaveToJsonPressed();
-                  },
-                ),
-                enabled: storagePermissionIsGranted,
-              ),
-              if (alwaysExportEnabled) SizedBox(height: tileSpacing),
-              if (alwaysExportEnabled)
-                ListTile(
-                  title: Text('Change export directory'),
-                  subtitle: Text(
-                    context
-                        .watch<SettingsProvider>()
-                        .settings
-                        .exportDirectoryPath,
-                  ),
-                  onTap: () => onChangeExportDirectoryPressed(),
-                  trailing: Icon(Icons.arrow_forward_ios_rounded),
-                  enabled: storagePermissionIsGranted,
-                ),
             ],
           ),
         ),
@@ -123,54 +84,23 @@ class _SettingsPageState extends State<SettingsPage> {
     );
   }
 
-  void onJsonExportPressed() async {
+  void onActivateJsonExportPressed() async {
-    if (!await checkStoragePermission()) return;
+    if (!await PermissionService.storagePermissionStatus.isGranted) return;
     Storage.exportToJsonFile().then(showExportInfo);
   }
 
-  void onJsonImportPressed() async {
+  void onActivateJsonImportPressed() async {
-    if (!await checkStoragePermission()) return;
+    if (!await PermissionService.storagePermissionStatus.isGranted) return;
     Storage.importFromJsonFile().then(showImportInfo);
   }
 
-  void onAlwaysSaveToJsonPressed() async {
+  Future<void> checkStoragePermission() async {
-    if (context.read<SettingsProvider>().settings.alwaysExportEnabled) {
-      context.read<SettingsProvider>().setExportDirectoryPath('', silent: true);
-      context.read<SettingsProvider>().setAlwaysExportEnabled(false);
-      return;
-    }
-
-    if (!await PermissionService.storagePermissionStatus.isGranted) return;
-    final dir = await Storage.selectDirectoryPath();
-    if (dir.isEmpty || !context.mounted) return;
-
-    // ignore: use_build_context_synchronously
-    context.read<SettingsProvider>().setExportDirectoryPath(dir, silent: true);
-
-    // ignore: use_build_context_synchronously
-    Storage.saveDataToFile().whenComplete(
-      // ignore: use_build_context_synchronously
-      () => context.read<SettingsProvider>().setAlwaysExportEnabled(true),
-    );
-  }
-
-  void onChangeExportDirectoryPressed() async {
-    if (!await PermissionService.storagePermissionStatus.isGranted) return;
-    final dir = await Storage.selectDirectoryPath();
-    if (dir.isEmpty || !context.mounted) return;
-
-    // ignore: use_build_context_synchronously
-    context.read<SettingsProvider>().setExportDirectoryPath(dir);
-  }
-
-  Future<bool> checkStoragePermission() async {
     PermissionService.storagePermissionStatus.then((value) {
+      storagePermissionIsGranted = value.isGranted;
       if (context.mounted && value.isGranted != storagePermissionIsGranted) {
-        storagePermissionIsGranted = value.isGranted;
         setState(() {});
       }
     });
-    return storagePermissionIsGranted;
   }
 
   void showExportInfo(bool success) => Notifying.showSnackbar(

lib/service/json_file_service.dart

@@ -14,10 +14,21 @@ class JsonFileService {
     required List<Bookmark> bookmarks,
   }) async {
     try {
-      final dir = await selectDirectoryPath();
+      final dir = await _directoryPath;
       if (dir.isEmpty) return false;
 
-      saveDataToFile(collections, bookmarks, dir);
+      final data = {
+        'collections': collections.map((c) => c.toJson()).toList(),
+        'bookmarks': bookmarks.map((b) => b.toJson()).toList(),
+      };
+      final json = jsonEncode(data).codeUnits;
+      final file = XFile.fromData(
+        Uint8List.fromList(json),
+        mimeType: 'application/json',
+        name: constants.jsonFileName,
+      );
+
+      file.saveTo('$dir/${constants.jsonFileName}');
     } catch (e) {
       return false;
     }
@@ -54,31 +65,7 @@ class JsonFileService {
     }
   }
 
-  static Future<bool> saveDataToFile(
+  static Future<String> get _directoryPath async {
-    List<Collection> collections,
-    List<Bookmark> bookmarks,
-    String directory,
-  ) async {
-    try {
-      final data = jsonEncode({
-        'collections': collections.map((c) => c.toJson()).toList(),
-        'bookmarks': bookmarks.map((b) => b.toJson()).toList(),
-      }).codeUnits;
-
-      final file = XFile.fromData(
-        Uint8List.fromList(data),
-        mimeType: 'application/json',
-        name: constants.jsonFileName,
-      );
-
-      file.saveTo('$directory/${constants.jsonFileName}');
-    } catch (e) {
-      return false;
-    }
-    return true;
-  }
-
-  static Future<String> selectDirectoryPath() async {
     if (Platform.isAndroid) {
       return await getDirectoryPath(
             initialDirectory: constants.defaultAndroidExportDirectory,

lib/service/settings_provider.dart

@@ -1,24 +0,0 @@
-import 'package:flutter/material.dart';
-
-import '../model/settings.dart';
-import 'storage.dart';
-
-class SettingsProvider extends ChangeNotifier {
-  SettingsProvider() : _settings = Storage.loadSettings();
-
-  Settings _settings;
-
-  Settings get settings => _settings;
-
-  void setExportDirectoryPath(String path, {bool silent = false}) {
-    _settings = _settings.copyWith(exportDirectoryPath: path);
-    Storage.saveSettings(_settings);
-    if (!silent) notifyListeners();
-  }
-
-  void setAlwaysExportEnabled(bool enabled, {bool silent = false}) {
-    _settings = _settings.copyWith(alwaysExportEnabled: enabled);
-    Storage.saveSettings(_settings);
-    if (!silent) notifyListeners();
-  }
-}

lib/service/storage.dart

@@ -4,49 +4,22 @@ import 'package:shared_preferences/shared_preferences.dart';
 
 import '../model/bookmark.dart';
 import '../model/collection.dart';
-import '../model/settings.dart';
 import 'json_file_service.dart';
 
 class Storage {
   static const String _bookmarksKey = 'bookmarks';
   static const String _collectionsKey = 'collections';
   static SharedPreferencesWithCache? _prefsWithCache;
-  static const String _settingsKey = 'settings';
   static const String _statsKey = 'stats';
-  static Settings _currentSettings = Settings.defaults();
 
   static Future<void> initialize() async {
     _prefsWithCache = await SharedPreferencesWithCache.create(
       cacheOptions: const SharedPreferencesWithCacheOptions(
-        allowList: <String>{
+        allowList: <String>{_collectionsKey, _bookmarksKey, _statsKey},
-          _collectionsKey,
-          _bookmarksKey,
-          _statsKey,
-          _settingsKey,
-        },
       ),
     );
   }
 
-  static Settings loadSettings() {
-    final jsonString = _prefs.getString(_settingsKey);
-    if (jsonString != null) {
-      final json = jsonDecode(jsonString) as Map<String, dynamic>;
-      _currentSettings = Settings.fromJson(json);
-      return _currentSettings;
-    } else {
-      final settings = Settings.defaults();
-      saveSettings(settings);
-      return settings;
-    }
-  }
-
-  static Future<void> saveSettings(Settings settings) {
-    final json = jsonEncode(settings.toJson());
-    _currentSettings = settings;
-    return _prefs.setString(_settingsKey, json);
-  }
-
   static List<Collection> loadCollections() {
     final jsonString = _prefs.getString(_collectionsKey) ?? '[]';
     final jsonList = jsonDecode(jsonString) as List;
@@ -66,13 +39,11 @@ class Storage {
   static Future<void> saveCollections(List<Collection> collections) async {
     final jsonList = collections.map((c) => c.toJson()).toList();
     await _prefs.setString(_collectionsKey, jsonEncode(jsonList));
-    if (_currentSettings.alwaysExportEnabled) saveDataToFile();
   }
 
   static Future<void> saveBookmarks(List<Bookmark> bookmarks) async {
     final jsonList = bookmarks.map((b) => b.toJson()).toList();
     await _prefs.setString(_bookmarksKey, jsonEncode(jsonList));
-    if (_currentSettings.alwaysExportEnabled) saveDataToFile();
   }
 
   static List<Bookmark> loadBookmarksForCollection(int collectionId) {
@@ -185,6 +156,15 @@ class Storage {
     await _prefs.setString(_statsKey, jsonEncode(stats));
   }
 
+  static SharedPreferencesWithCache get _prefs {
+    if (_prefsWithCache == null) {
+      throw StateError(
+        'BookmarkStorage not initialized. Call initialize() first.',
+      );
+    }
+    return _prefsWithCache!;
+  }
+
   static Future<bool> exportToJsonFile() => JsonFileService.exportToJson(
     collections: loadCollections(),
     bookmarks: loadBookmarks(),
@@ -200,22 +180,4 @@ class Storage {
     }
     return false;
   }
-
-  static Future<String> selectDirectoryPath() =>
-      JsonFileService.selectDirectoryPath();
-
-  static Future<bool> saveDataToFile() => JsonFileService.saveDataToFile(
-    loadCollections(),
-    loadBookmarks(),
-    _currentSettings.exportDirectoryPath,
-  );
-
-  static SharedPreferencesWithCache get _prefs {
-    if (_prefsWithCache == null) {
-      throw StateError(
-        'BookmarkStorage not initialized. Call initialize() first.',
-      );
-    }
-    return _prefsWithCache!;
-  }
 }

lib/theme.dart

@@ -22,6 +22,9 @@ ThemeData _baseTheme(ColorScheme scheme) =>
         shape: RoundedRectangleBorder(
           borderRadius: BorderRadiusGeometry.circular(12),
         ),
+        textColor: scheme.onPrimaryContainer,
+        selectedTileColor: scheme.primaryContainer,
+        selectedColor: scheme.onPrimaryContainer,
         contentPadding: EdgeInsetsDirectional.only(start: 16.0, end: 24.0),
       ),
     );

lib/widgets/collection_page_widgets/list_item_actions_widget.dart

@@ -0,0 +1,53 @@
+import 'package:flutter/material.dart';
+
+class ListItemActionsWidget extends StatelessWidget {
+  final VoidCallback _onDeletePressed;
+  final VoidCallback _onCancelPressed;
+  final VoidCallback _onEditPressed;
+
+  const ListItemActionsWidget({
+    super.key,
+    required void Function() onDeletePressed,
+    required void Function() onCancelPressed,
+    required void Function() onEditPressed,
+  }) : _onEditPressed = onEditPressed,
+       _onCancelPressed = onCancelPressed,
+       _onDeletePressed = onDeletePressed;
+
+  @override
+  Widget build(BuildContext context) {
+    return Container(
+      padding: EdgeInsetsGeometry.fromLTRB(
+        10,
+        10,
+        10,
+        MediaQuery.of(context).viewPadding.bottom,
+      ),
+      color: Theme.of(context).colorScheme.surfaceContainer,
+      child: Row(
+        mainAxisAlignment: MainAxisAlignment.spaceBetween,
+        crossAxisAlignment: CrossAxisAlignment.start,
+        children: [
+          IconButton(
+            onPressed: _onCancelPressed,
+            icon: const Icon(Icons.close_rounded),
+          ),
+          Row(
+            mainAxisAlignment: MainAxisAlignment.end,
+            mainAxisSize: MainAxisSize.min,
+            children: [
+              IconButton(
+                onPressed: _onDeletePressed,
+                icon: const Icon(Icons.delete_forever_rounded),
+              ),
+              IconButton(
+                onPressed: _onEditPressed,
+                icon: const Icon(Icons.edit_rounded),
+              ),
+            ],
+          ),
+        ],
+      ),
+    );
+  }
+}