chore: Follow up new dialogs

krille-chan · Dec 10, 2024 · 21be49c · 21be49c
1 parent a169484
commit 21be49c
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 3 deletions.
diff --git a/lib/utils/adaptive_bottom_sheet.dart b/lib/utils/adaptive_bottom_sheet.dart
@@ -35,7 +35,6 @@ Future<T?> showAdaptiveBottomSheet<T>({
       maxWidth: FluffyThemes.columnWidth * 1.25,
     ),
     backgroundColor: Colors.transparent,
-    showDragHandle: !dialogMode,
     clipBehavior: Clip.hardEdge,
   );
 }
diff --git a/lib/widgets/adaptive_dialogs/adaptive_dialog_action.dart b/lib/widgets/adaptive_dialogs/adaptive_dialog_action.dart
@@ -3,12 +3,14 @@ import 'package:flutter/material.dart';
 
 class AdaptiveDialogAction extends StatelessWidget {
   final VoidCallback? onPressed;
+  final bool autofocus;
   final Widget child;
 
   const AdaptiveDialogAction({
     super.key,
     required this.onPressed,
     required this.child,
+    this.autofocus = false,
   });
 
   @override
@@ -19,10 +21,18 @@ class AdaptiveDialogAction extends StatelessWidget {
       case TargetPlatform.fuchsia:
       case TargetPlatform.linux:
       case TargetPlatform.windows:
-        return TextButton(onPressed: onPressed, child: child);
+        return TextButton(
+          onPressed: onPressed,
+          autofocus: autofocus,
+          child: child,
+        );
       case TargetPlatform.iOS:
       case TargetPlatform.macOS:
-        return CupertinoDialogAction(onPressed: onPressed, child: child);
+        return CupertinoDialogAction(
+          onPressed: onPressed,
+          isDefaultAction: autofocus,
+          child: child,
+        );
     }
   }
 }
diff --git a/lib/widgets/adaptive_dialogs/show_ok_cancel_alert_dialog.dart b/lib/widgets/adaptive_dialogs/show_ok_cancel_alert_dialog.dart
@@ -36,6 +36,7 @@ Future<OkCancelResult?> showOkCancelAlertDialog({
           AdaptiveDialogAction(
             onPressed: () =>
                 Navigator.of(context).pop<OkCancelResult>(OkCancelResult.ok),
+            autofocus: true,
             child: Text(
               okLabel ?? L10n.of(context).ok,
               style: isDestructive
@@ -70,6 +71,7 @@ Future<OkCancelResult?> showOkAlertDialog({
           AdaptiveDialogAction(
             onPressed: () =>
                 Navigator.of(context).pop<OkCancelResult>(OkCancelResult.ok),
+            autofocus: true,
             child: Text(okLabel ?? L10n.of(context).close),
           ),
         ],

diff --git a/lib/widgets/adaptive_dialogs/show_text_input_dialog.dart b/lib/widgets/adaptive_dialogs/show_text_input_dialog.dart
@@ -87,6 +87,7 @@ Future<String?> showTextInputDialog({
                   }
                   Navigator.of(context).pop<String>(input);
                 },
+                autofocus: true,
                 child: Text(
                   okLabel ?? L10n.of(context).ok,
                   style: isDestructive