From a2ff2a3788bbac9d08683373538540e942fffcd7 Mon Sep 17 00:00:00 2001 From: Ming Ming Date: Sat, 10 Jul 2021 22:31:57 +0800 Subject: [PATCH] Move "delete selected" into menus --- lib/widget/home_photos.dart | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/lib/widget/home_photos.dart b/lib/widget/home_photos.dart index 034bc2dd..538419de 100644 --- a/lib/widget/home_photos.dart +++ b/lib/widget/home_photos.dart @@ -166,13 +166,6 @@ class _HomePhotosState extends State _onSelectionAppBarAddToAlbumPressed(context); }, ), - IconButton( - icon: const Icon(Icons.delete), - tooltip: AppLocalizations.of(context).deleteSelectedTooltip, - onPressed: () { - _onSelectionAppBarDeletePressed(context); - }, - ), PopupMenuButton( tooltip: MaterialLocalizations.of(context).moreButtonTooltip, itemBuilder: (context) => [ @@ -181,6 +174,10 @@ class _HomePhotosState extends State child: Text(AppLocalizations.of(context).archiveSelectedMenuLabel), ), + PopupMenuItem( + value: _SelectionAppBarMenuOption.delete, + child: Text(AppLocalizations.of(context).deleteSelectedTooltip), + ), ], onSelected: (option) { _onSelectionAppBarMenuSelected(context, option); @@ -387,6 +384,10 @@ class _HomePhotosState extends State _onSelectionAppBarArchivePressed(context); break; + case _SelectionAppBarMenuOption.delete: + _onSelectionAppBarDeletePressed(context); + break; + default: _log.shout("[_onSelectionAppBarMenuSelected] Unknown option: $option"); break;