fix: Prevent multiple Tap when importing Playlist

khaled-0 · Dec 2, 2024 · 368679e · 368679e
1 parent 00f609a
commit 368679e
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 7 deletions.
diff --git a/lib/app/home/home_screen.dart b/lib/app/home/home_screen.dart
@@ -98,6 +98,7 @@ class _HomeTabState extends State<HomeTab> {
     showDialog(
       context: context,
       useRootNavigator: true,
+      barrierDismissible: false,
       builder: (_) => ChangeNotifierProvider.value(
         value: context.read<LibraryProvider>(),
         child: ImportPlaylistDialog(url: url),

diff --git a/lib/app/library/import_playlist_dialog.dart b/lib/app/library/import_playlist_dialog.dart
@@ -19,6 +19,7 @@ class _ImportPlaylistDialogState extends State<ImportPlaylistDialog> {
   String? error;
 
   Future<void> tryImportPlaylist() async {
+    if (loading) return;
     try {
       error = null;
       setState(() => loading = true);
@@ -95,8 +96,9 @@ class _ImportPlaylistDialogState extends State<ImportPlaylistDialog> {
                       error!,
                       maxLines: 3,
                       overflow: TextOverflow.ellipsis,
-                      style:
-                          TextStyle(color: Theme.of(context).colorScheme.error),
+                      style: TextStyle(
+                        color: Theme.of(context).colorScheme.error,
+                      ),
                     ),
                   ),
                 ),
@@ -105,11 +107,16 @@ class _ImportPlaylistDialogState extends State<ImportPlaylistDialog> {
         ),
       ),
       actions: [
-        TextButton(
-          onPressed: () => Navigator.pop(context),
-          child: const Text("Cancel"),
-        ),
-        FilledButton(onPressed: tryImportPlaylist, child: const Text("Import"))
+        if (!loading)
+          TextButton(
+            onPressed: () => Navigator.pop(context),
+            child: const Text("Cancel"),
+          ),
+        if (!loading)
+          FilledButton(
+            onPressed: tryImportPlaylist,
+            child: const Text("Import"),
+          ),
       ],
     );
   }

diff --git a/lib/app/library/library_tab.dart b/lib/app/library/library_tab.dart
@@ -101,6 +101,7 @@ class LibraryTab extends StatelessWidget {
     void onPressed() => showDialog(
           context: context,
           useRootNavigator: true,
+          barrierDismissible: false,
           builder: (_) => ChangeNotifierProvider.value(
             value: context.read<LibraryProvider>(),
             child: const ImportPlaylistDialog(),