TerribleDev/ente
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: selection update logic

Browse Source
This commit is contained in:
Prateek Sunal
2025-07-26 15:41:32 +05:30
parent de481cc689
commit 58baa04df3
2 changed files with 13 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
mobile/apps/photos/lib/ui/collections/album/smart_album_people.dart
View File

@@ -54,10 +54,6 @@ class _SmartAlbumPeopleState extends State<SmartAlbumPeople> {
@override
Widget build(BuildContext context) {
final areIdsChanged = currentConfig?.personIDs != null
? !setEquals(_selectedPeople.personIds, currentConfig!.personIDs)
: _selectedPeople.personIds.isNotEmpty;
return Scaffold(
bottomNavigationBar: Padding(
padding: EdgeInsets.fromLTRB(
@@ -69,6 +65,12 @@ class _SmartAlbumPeopleState extends State<SmartAlbumPeople> {
child: ListenableBuilder(
listenable: _selectedPeople,
builder: (context, _) {
final areIdsChanged = currentConfig?.personIDs != null
? !setEquals(
_selectedPeople.personIds,
currentConfig!.personIDs,
)
: _selectedPeople.personIds.isEmpty;
return ButtonWidget(
buttonType: ButtonType.primary,
buttonSize: ButtonSize.large,

11
mobile/apps/photos/lib/ui/settings/widgets/people_widget_settings.dart
View File

@@ -48,10 +48,6 @@ class _PeopleWidgetSettingsState extends State<PeopleWidgetSettings> {
@override
Widget build(BuildContext context) {
final areIdsChanged = lastSelectedPeople != null
? !setEquals(_selectedPeople.personIds, lastSelectedPeople)
: _selectedPeople.personIds.isNotEmpty;
return Scaffold(
bottomNavigationBar: hasInstalledAny
? Padding(
@@ -64,6 +60,13 @@ class _PeopleWidgetSettingsState extends State<PeopleWidgetSettings> {
child: ListenableBuilder(
listenable: _selectedPeople,
builder: (context, _) {
final areIdsChanged = lastSelectedPeople != null
? !setEquals(
_selectedPeople.personIds,
lastSelectedPeople,
)
: _selectedPeople.personIds.isEmpty;
return ButtonWidget(
buttonType: ButtonType.primary,
buttonSize: ButtonSize.large,