diff --git a/lib/screens/creators/sites/site_list.dart b/lib/screens/creators/sites/site_list.dart index f468fd6a..b980c8f7 100644 --- a/lib/screens/creators/sites/site_list.dart +++ b/lib/screens/creators/sites/site_list.dart @@ -167,22 +167,10 @@ class _CreatorSiteItem extends HookConsumerWidget { ], ), onTap: () async { - final confirmed = await showDialog( - context: context, - builder: (context) => AlertDialog( - title: Text('deleteSite'.tr()), - content: Text('deleteSiteConfirm'.tr()), - actions: [ - TextButton( - onPressed: () => Navigator.of(context).pop(false), - child: Text('cancel'.tr()), - ), - TextButton( - onPressed: () => Navigator.of(context).pop(true), - child: Text('delete'.tr()), - ), - ], - ), + final confirmed = await showConfirmAlert( + 'publicationSiteDeleteConfirm'.tr(), + 'deleteSite'.tr(), + isDanger: true, ); if (confirmed == true) { try { diff --git a/lib/widgets/sites/site_action_menu.dart b/lib/widgets/sites/site_action_menu.dart index 64fcae4f..8e74a530 100644 --- a/lib/widgets/sites/site_action_menu.dart +++ b/lib/widgets/sites/site_action_menu.dart @@ -19,64 +19,50 @@ class SiteActionMenu extends HookConsumerWidget { @override Widget build(BuildContext context, WidgetRef ref) { return PopupMenuButton( - itemBuilder: - (context) => [ - PopupMenuItem( - value: 'edit', - child: Row( - children: [ - Icon( - Symbols.edit, - color: Theme.of(context).colorScheme.onSurface, - ), - const Gap(16), - Text('edit'.tr()), - ], + itemBuilder: (context) => [ + PopupMenuItem( + value: 'edit', + child: Row( + children: [ + Icon( + Symbols.edit, + color: Theme.of(context).colorScheme.onSurface, ), - ), - const PopupMenuDivider(), - PopupMenuItem( - value: 'delete', - child: Row( - children: [ - const Icon(Symbols.delete, color: Colors.red), - const Gap(16), - Text('delete'.tr()).textColor(Colors.red), - ], - ), - ), - ], + const Gap(16), + Text('edit'.tr()), + ], + ), + ), + const PopupMenuDivider(), + PopupMenuItem( + value: 'delete', + child: Row( + children: [ + const Icon(Symbols.delete, color: Colors.red), + const Gap(16), + Text('delete'.tr()).textColor(Colors.red), + ], + ), + ), + ], onSelected: (value) async { switch (value) { case 'edit': showModalBottomSheet( context: context, isScrollControlled: true, - builder: - (context) => SiteForm(pubName: pubName, siteSlug: site.slug), + builder: (context) => + SiteForm(pubName: pubName, siteSlug: site.slug), ).then((_) { // Refresh site data after potential edit ref.invalidate(publicationSiteDetailProvider(pubName, site.slug)); }); break; case 'delete': - final confirmed = await showDialog( - context: context, - builder: - (context) => AlertDialog( - title: Text('deleteSite'.tr()), - content: Text('publicationSiteDeleteConfirm'.tr()), - actions: [ - TextButton( - onPressed: () => Navigator.of(context).pop(false), - child: Text('cancel'.tr()), - ), - TextButton( - onPressed: () => Navigator.of(context).pop(true), - child: Text('delete'.tr()), - ), - ], - ), + final confirmed = await showConfirmAlert( + 'publicationSiteDeleteConfirm'.tr(), + 'deleteSite'.tr(), + isDanger: true, ); if (confirmed == true) {