From fb4c12d1a5a29dc51f49496f543fd6676c327839 Mon Sep 17 00:00:00 2001
From: Rongjian Zhang <pd4d10@gmail.com>
Date: Thu, 19 Sep 2019 20:50:47 +0800
Subject: [PATCH] feat: adapter picker for material theme

---
 lib/models/theme.dart | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/lib/models/theme.dart b/lib/models/theme.dart
index 2882519..a5a9c41 100644
--- a/lib/models/theme.dart
+++ b/lib/models/theme.dart
@@ -190,12 +190,12 @@ class ThemeModel with ChangeNotifier {
 
   static Timer _debounce;
 
-  Future<T> showPicker<T>(
+  showPicker<T>(
     BuildContext context, {
     @required T initialValue,
     @required List<PickerItem<T>> items,
     @required Function(T item) onChange,
-  }) {
+  }) async {
     switch (theme) {
       case AppThemeMap.cupertino:
         return showCupertinoModalPopup<T>(
@@ -220,12 +220,18 @@ class ThemeModel with ChangeNotifier {
           },
         );
       default:
-        return showModalBottomSheet<T>(
+        final value = await showMenu<T>(
           context: context,
-          builder: (context) {
-            return null; // TODO:
-          },
+          initialValue: initialValue,
+          items: items
+              .map((item) =>
+                  PopupMenuItem(value: item.value, child: Text(item.text)))
+              .toList(),
+          position: RelativeRect.fill,
         );
+        if (value != null) {
+          onChange(value);
+        }
     }
   }
 }