From 35aed07bc6ee744aa4bb843ab8bf2f86a4bede5d Mon Sep 17 00:00:00 2001 From: laurenspriem Date: Fri, 3 May 2024 16:59:51 +0530 Subject: [PATCH] [mob][photos] Rename "remove" to "remove label" for person --- mobile/lib/ui/viewer/people/people_app_bar.dart | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/mobile/lib/ui/viewer/people/people_app_bar.dart b/mobile/lib/ui/viewer/people/people_app_bar.dart index 545437048..0755dcf0b 100644 --- a/mobile/lib/ui/viewer/people/people_app_bar.dart +++ b/mobile/lib/ui/viewer/people/people_app_bar.dart @@ -44,7 +44,7 @@ class PeopleAppBar extends StatefulWidget { enum PeoplPopupAction { rename, setCover, - remove, + removeLabel, viewPhotos, confirmPhotos, unignore, @@ -171,15 +171,15 @@ class _AppBarWidgetState extends State { // ), // ), - PopupMenuItem( - value: PeoplPopupAction.remove, + const PopupMenuItem( + value: PeoplPopupAction.removeLabel, child: Row( children: [ - const Icon(Icons.remove_circle_outline), - const Padding( + Icon(Icons.remove_circle_outline), + Padding( padding: EdgeInsets.all(8), ), - Text(S.of(context).remove), + Text("Remove label"), ], ), ), @@ -260,7 +260,7 @@ class _AppBarWidgetState extends State { await setCoverPhoto(context); } else if (value == PeoplPopupAction.unignore) { await _showPerson(context); - } else if (value == PeoplPopupAction.remove) { + } else if (value == PeoplPopupAction.removeLabel) { await PersonService.instance.deletePerson(widget.person.remoteID); } }, @@ -275,7 +275,8 @@ class _AppBarWidgetState extends State { bool assignName = false; await showChoiceDialog( context, - title: "Are you sure you want to show this person in people section again??", + title: + "Are you sure you want to show this person in people section again??", firstButtonLabel: "Yes, show person", firstButtonOnTap: () async { try {