Skip to content

Album (and some settings) improvements#314

Merged
marksam32 merged 6 commits intomasterfrom
album_improvements
May 7, 2026
Merged

Album (and some settings) improvements#314
marksam32 merged 6 commits intomasterfrom
album_improvements

Conversation

@MikaeldeVerdier
Copy link
Copy Markdown
Contributor

Fixed select and download images. Improved image browser by now showing thumbnail while loading and preloading adjacent images. Reduced amount of futures used and implemented an images_service singleton to fetch all images. Fixed "unsaved changes" dialog box in settings.

…r selectAll button container; changed color of select textbutton (in appBar)
…h improved and simplified caching); improved imageBrowser to show thumbnail while loading fullsize image as well as keeping pages alive for longer; reduced usage of futures; minor improvements
@MikaeldeVerdier MikaeldeVerdier requested a review from marksam32 May 4, 2026 16:17
@MikaeldeVerdier MikaeldeVerdier added bug Something isn't working enhancement New feature or request labels May 4, 2026
Copy link
Copy Markdown
Contributor

@marksam32 marksam32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall good! Just these things that I found

Comment thread lib/screens/settings/settings.dart
Comment thread lib/services/images.service.dart Outdated
@MikaeldeVerdier
Copy link
Copy Markdown
Contributor Author

Good catch, should be fixed now!

Copy link
Copy Markdown
Contributor

@marksam32 marksam32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@marksam32 marksam32 merged commit 51611d0 into master May 7, 2026
1 check passed
@marksam32 marksam32 deleted the album_improvements branch May 7, 2026 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants