Skip to content

Code Quality: Added thumbnail cache settings UI#18176

Merged
yair100 merged 5 commits intofiles-community:mainfrom
workbysaran:sg/feature-thumbnailcache-settings
Apr 26, 2026
Merged

Code Quality: Added thumbnail cache settings UI#18176
yair100 merged 5 commits intofiles-community:mainfrom
workbysaran:sg/feature-thumbnailcache-settings

Conversation

@workbysaran
Copy link
Copy Markdown
Contributor

Feature: Added thumbnail cache settings UI.

Brought over the thumbnail cache enable/disable toggle, cache size limit setting and clear cache button U from 'sg/feature-thumbnail-cache'.

Steps used to test these changes

  1. Open Settings > Advanced
  2. Verify the "Cache thumbnails" toggle and expandable section appear
  3. Toggle enable/disable and confirm cache size limit input works
  4. Verify Clear Cache button is present

Comment thread src/Files.App/Views/Settings/AdvancedPage.xaml Outdated
@yair100 yair100 added the changes requested Changes are needed for this pull request label Feb 17, 2026
@yair100 yair100 changed the title Feature: Added thumbnail cache settings UI Code Quality: Added thumbnail cache settings UI Feb 17, 2026
@yair100 yair100 force-pushed the sg/feature-thumbnailcache-settings branch from 50d538d to a961e5b Compare April 26, 2026 21:34
yair100
yair100 previously approved these changes Apr 26, 2026
Copy link
Copy Markdown
Member

@yair100 yair100 left a comment

Choose a reason for hiding this comment

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

LGTM

@yair100 yair100 added ready to merge Pull requests that are approved and ready to merge and removed changes requested Changes are needed for this pull request labels Apr 26, 2026
@yair100 yair100 merged commit cf434d3 into files-community:main Apr 26, 2026
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Pull requests that are approved and ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants