Skip to content

Gallery mode and screenshot cache - #574

Merged
jackpot51 merged 5 commits into
pop-os:masterfrom
hojjatabdollahi:hojjat/gallery
Oct 5, 2026
Merged

jackpot51 merged 5 commits into
pop-os:masterfrom
hojjatabdollahi:hojjat/gallery

Conversation

@hojjatabdollahi

@hojjatabdollahi hojjatabdollahi commented Jul 15, 2026 •

Copy link
Copy Markdown
Contributor

This pulls in gallery mode and large image decode offloading from cosmic-files, and the image cache from #456. Image cache is limited to 250mb max (from @FreddyFunk's PR), I only changed the cache-key to take app version into account.

  • you can enlarge an image by clicking on it (gallery view)
  • caches the images, so it's faster next time
  • caches using image url and app version, so refetches images on app update
  • removed the next/prev buttons if there's only one image
recording-2026-07-15_12-52-11-trimmed.mp4

This looks like a big PR, but it's mostly either from cosmic-files, or from Freddy's PR.


  • I have disclosed use of any AI generated code in my commit messages.
    • If you are using an LLM, and do not fully understand the changes it is making to the code base, do not create a PR.
    • In our experience, AI generated code often results in overly complex code that lacks enough context for a proper fix or feature inclusion. This results in considerably longer code reviews. Due to this, AI authored or partially authored PRs may be closed without comment.
  • I understand these changes in full and will be able to respond to review comments.
  • My change is accurately described in the commit message.
  • My contribution is tested and working as described.
  • I have read the Developer Certificate of Origin and certify my contribution under its conditions.

@jklgrasso

Copy link
Copy Markdown

Please resolve the conflicts when you're able

@hojjatabdollahi

Copy link
Copy Markdown
Contributor Author

@jklgrasso done

@jklgrasso jklgrasso left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Regression testing passed ## Checklist
  • Able to install & remove a Flatpak app.
  • Able to install & remove a Deb app.
  • Able to install Deb updates.
  • Searching still works.
  • Browsing categories still works.
  • Able to remove & re-add Flatpak sources.

@jackpot51
jackpot51 merged commit 4ff1685 into pop-os:master Oct 5, 2026
6 checks passed
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.

4 participants