feat: use popup menu button for material

This commit is contained in:
Rongjian Zhang 2019-09-30 16:28:55 +08:00
parent 9c4cd73814
commit 951c50c2a9
1 changed files with 24 additions and 5 deletions

View File

@ -1,7 +1,6 @@
import 'package:flutter/material.dart';
import 'package:flutter/cupertino.dart';
import 'package:git_touch/utils/utils.dart';
import 'package:git_touch/widgets/action_entry.dart';
import 'package:provider/provider.dart';
import 'package:git_touch/models/theme.dart';
import 'package:share/share.dart';
@ -9,8 +8,14 @@ import 'package:share/share.dart';
class ActionItem {
String text;
void Function() onPress;
IconData iconData;
ActionItem({
@required this.text,
@required this.onPress,
this.iconData,
});
ActionItem({@required this.text, @required this.onPress});
ActionItem.share(String url)
: text = 'Share',
onPress = (() {
@ -38,9 +43,8 @@ class ActionButton extends StatelessWidget {
Widget build(BuildContext context) {
switch (Provider.of<ThemeModel>(context).theme) {
case AppThemeType.cupertino:
default:
return ActionEntry(
iconData: iconData,
return GestureDetector(
child: Icon(iconData, size: 22),
onTap: () async {
var value = await showCupertinoModalPopup<int>(
context: context,
@ -71,6 +75,21 @@ class ActionButton extends StatelessWidget {
}
},
);
default:
return PopupMenuButton(
icon: Icon(iconData),
itemBuilder: (context) {
return items.asMap().entries.map((entry) {
return PopupMenuItem(
value: entry.key,
child: Text(entry.value.text),
);
}).toList();
},
onSelected: (selected) {
items[selected].onPress();
},
);
}
}
}