diff --git a/mobile/lib/presentation/widgets/asset_viewer/viewer_kebab_menu.widget.dart b/mobile/lib/presentation/widgets/asset_viewer/viewer_kebab_menu.widget.dart index b740df7f37..4651b5eea8 100644 --- a/mobile/lib/presentation/widgets/asset_viewer/viewer_kebab_menu.widget.dart +++ b/mobile/lib/presentation/widgets/asset_viewer/viewer_kebab_menu.widget.dart @@ -8,9 +8,7 @@ import 'package:immich_mobile/presentation/widgets/action_buttons/base_action_bu import 'package:immich_mobile/providers/infrastructure/asset_viewer/current_asset.provider.dart'; class ViewerKebabMenu extends ConsumerWidget { - final VoidCallback onConfigureButtons; - - const ViewerKebabMenu({super.key, required this.onConfigureButtons}); + const ViewerKebabMenu({super.key}); @override Widget build(BuildContext context, WidgetRef ref) { @@ -19,37 +17,29 @@ class ViewerKebabMenu extends ConsumerWidget { return const SizedBox.shrink(); } - final theme = context.themeData; final menuChildren = [ BaseActionButton( - label: 'open_bottom_sheet_info'.tr(), + label: 'about'.tr(), iconData: Icons.info_outline, + menuItem: true, onPressed: () => EventStream.shared.emit(const ViewerOpenBottomSheetEvent()), ), - const Divider(height: 0), - BaseActionButton(label: 'reorder_buttons'.tr(), iconData: Icons.tune, onPressed: onConfigureButtons), ]; return MenuAnchor( + consumeOutsideTap: true, style: MenuStyle( - backgroundColor: WidgetStatePropertyAll(theme.scaffoldBackgroundColor), + backgroundColor: WidgetStatePropertyAll(context.themeData.scaffoldBackgroundColor), elevation: const WidgetStatePropertyAll(4), shape: const WidgetStatePropertyAll( RoundedRectangleBorder(borderRadius: BorderRadius.all(Radius.circular(12))), ), - padding: const WidgetStatePropertyAll(EdgeInsets.symmetric(vertical: 2)), ), menuChildren: menuChildren, builder: (context, controller, child) { return IconButton( icon: const Icon(Icons.more_vert_rounded), - onPressed: () { - if (controller.isOpen) { - controller.close(); - } else { - controller.open(); - } - }, + onPressed: () => controller.isOpen ? controller.close() : controller.open(), ); }, );