Fix MacosPulldownMenuItem.onTap doesn't open alert dialog (#520)

Author: TahaTesser

CHANGELOG.md

@@ -1,3 +1,6 @@
+## [2.1.1]
+* Fixed a bug where `MacosPulldownMenuItem` would not show an alert dialog when tapped.
+
 ## [2.1.0]
 * Updated dependencies
 * Support macOS 15

lib/src/buttons/pulldown_button.dart

@@ -65,8 +65,8 @@ class _MacosPulldownMenuItemButtonState
     final MacosPulldownMenuEntry menuEntity =
         widget.route.items[widget.itemIndex].item!;
     if (menuEntity is MacosPulldownMenuItem) {
+      Navigator.of(context).pop();
       menuEntity.onTap?.call();
-      Navigator.pop(context);
     }
   }
 

pubspec.yaml

@@ -1,6 +1,6 @@
 name: macos_ui
 description: Flutter widgets and themes implementing the current macOS design language.
-version: 2.1.0
+version: 2.1.1
 homepage: "https://macosui.dev"
 repository: "https://github.com/GroovinChip/macos_ui"
 

test/buttons/pulldown_button_test.dart

@@ -180,5 +180,59 @@ void main() {
         ],
       );
     });
+
+    testWidgets(
+      'MacosPulldownMenuItem.onTap shows alert dialog',
+      (WidgetTester tester) async {
+        await tester.pumpWidget(
+          MacosApp(
+            home: MacosWindow(
+              child: MacosScaffold(
+                children: [
+                  ContentArea(
+                    builder: (context, _) {
+                      return Center(
+                          child: MacosPulldownButton(
+                        title: "test",
+                        items: [
+                          MacosPulldownMenuItem(
+                            title: const Text('Open Alert Dialog'),
+                            onTap: () => showMacosAlertDialog(
+                              context: context,
+                              builder: (context) => MacosAlertDialog(
+                                appIcon: const MacosIcon(CupertinoIcons.eyedropper),
+                                title: const Text('Title'),
+                                message: const Text('Message'),
+                                primaryButton: PushButton(
+                                  controlSize: ControlSize.large,
+                                  onPressed: Navigator.of(context).pop,
+                                  child: const Text('Close'),
+                                ),
+                              ),
+                            ),
+                          ),
+                        ],
+                      ));
+                    },
+                  ),
+                ],
+              ),
+            ),
+          ),
+        );
+
+        // Tap the pulldown button.
+        await tester.tap(find.text('test'));
+        await tester.pumpAndSettle();
+        // Tap the menu item to show the alert dialog.
+        await tester.tap(find.text('Open Alert Dialog'));
+        await tester.pumpAndSettle();
+
+        expect(find.text('Open Alert Dialog'), findsNothing);
+        expect(find.text('Title'), findsOneWidget);
+        expect(find.text('Message'), findsOneWidget);
+        expect(find.text('Close'), findsOneWidget);
+      },
+    );
   });
 }