Skip to content

Add customizable thumbnail cache management#432

Closed
nicpadilla wants to merge 1 commit intoFossifyOrg:masterfrom
nicpadilla:master
Closed

Add customizable thumbnail cache management#432
nicpadilla wants to merge 1 commit intoFossifyOrg:masterfrom
nicpadilla:master

Conversation

@nicpadilla
Copy link

@nicpadilla nicpadilla commented Mar 28, 2025

What is it?

  • Bugfix
  • Feature
  • Codebase improvement

Description of the changes in your PR

Add user-configurable thumbnail cache size limits to give users more options for memory management. Glide's default thumbnail cache size is 250MB which is insufficient for larger galleries. This adds a new section in the settings menu to:

  • View current cache usage
  • Select from predefined cache size limits (250MB default, 500MB, 1GB, 2GB, 5GB)
  • Clear the thumbnail cache manually

Before/After Screenshots/Screen Record

  • Before:
Screenshot_before
  • After:
Screenshot_after_1 Screenshot_after_2

Fixes the following issue(s)

Relies on the following changes

Acknowledgement

Add user-configurable thumbnail cache size limits to give users
more options for memory management. Glide's default thumbnail cache size
is 250MB which is insufficient for larger galleries. This adds a new
section in the settings menu to:

- View current cache usage
- Select from predefined cache size limits
  (250MB default, 500MB, 1GB, 2GB, 5GB)
- Clear the thumbnail cache manually
@nicpadilla nicpadilla marked this pull request as ready for review March 28, 2025 03:10
@naveensingh
Copy link
Member

Nice but please read the contribution guidelines:

Before fixing a bug or adding a new feature, ensure it’s already reported or requested in the repository, open, and doesn’t have the needs triage label. If the issue is closed or has needs triage label, your PR will likely be rejected. The only exceptions are critical bugs that we weren't able to classify yet.

(Also, I'm curious. Did you use some agentic tool like Claude Code for this?)

Thanks!

@nicpadilla
Copy link
Author

Got it. Heh, I did use Cursor (Claude 3.7 in particular) to help. What was the giveaway?

Anyhow, anything I can do to get this triaged so this could be considered?

@naveensingh
Copy link
Member

Heh, I did use Cursor (Claude 3.7 in particular) to help. What was the giveaway?

Hehe. I got mad AI detection skills ;)

Anyhow, anything I can do to get this triaged so this could be considered?

Not really, it's just not planned for now. You may subscribe to the issue, I'll comment if/when it's ready for development.
Thanks, though.

@JamesPous
Copy link

JamesPous commented Aug 31, 2025

I would like to see this cache setting in one of the next releases, because my gallery app went to the 250 MB limit.
Is there a chance to see this feature in the future? Does the source code violate to the applicable rules?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Hope to add the option to set the cache upper limit

3 participants