diff --git a/mobile/apps/photos/lib/ui/collections/album/smart_album_people.dart b/mobile/apps/photos/lib/ui/collections/album/smart_album_people.dart index 5d798c635e..481a57f297 100644 --- a/mobile/apps/photos/lib/ui/collections/album/smart_album_people.dart +++ b/mobile/apps/photos/lib/ui/collections/album/smart_album_people.dart @@ -70,7 +70,7 @@ class _SmartAlbumPeopleState extends State { _selectedPeople.personIds, currentConfig!.personIDs, ) - : _selectedPeople.personIds.isEmpty; + : _selectedPeople.personIds.isNotEmpty; return ButtonWidget( buttonType: ButtonType.primary, buttonSize: ButtonSize.large, diff --git a/mobile/apps/photos/lib/ui/settings/widgets/people_widget_settings.dart b/mobile/apps/photos/lib/ui/settings/widgets/people_widget_settings.dart index c724672d83..10d1c55830 100644 --- a/mobile/apps/photos/lib/ui/settings/widgets/people_widget_settings.dart +++ b/mobile/apps/photos/lib/ui/settings/widgets/people_widget_settings.dart @@ -65,17 +65,16 @@ class _PeopleWidgetSettingsState extends State { _selectedPeople.personIds, lastSelectedPeople, ) - : _selectedPeople.personIds.isEmpty; + : _selectedPeople.personIds.isNotEmpty; return ButtonWidget( buttonType: ButtonType.primary, buttonSize: ButtonSize.large, labelText: S.of(context).save, shouldSurfaceExecutionStates: false, - isDisabled: areIdsChanged, + isDisabled: !areIdsChanged, onTap: areIdsChanged - ? null - : () async { + ? () async { unawaited( PeopleHomeWidgetService.instance .setSelectedPeople( @@ -83,7 +82,8 @@ class _PeopleWidgetSettingsState extends State { ), ); Navigator.pop(context); - }, + } + : null, ); }, ),