Repository navigation
feat: screenshots page with auto-clean - #1385
Merged
Merged
Conversation
sameerasw
marked this pull request as draft
October 15, 2026 12:24
jawzf
force-pushed
the
claude/screenshots-page
branch
from
October 15, 2026 15:56
97b4d5f to
b3c0508
Compare
jawzf
marked this pull request as ready for review
October 15, 2026 15:58
sameerasw
marked this pull request as draft
October 15, 2026 17:18
jawzf
force-pushed
the
claude/screenshots-page
branch
from
October 15, 2026 17:30
9162b6c to
a7b38bf
Compare
jawzf
marked this pull request as ready for review
October 15, 2026 17:30
Moves "Delete shared screenshots" onto a new Screenshots page under Security and adds: - a delay after sharing (immediately, 1 hour, 1 day) - auto-clean: every new screenshot is trashed after 1, 7 or 30 days, with a quiet notification offering Keep and Trash now; starred screenshots are kept - a count of cleaned screenshots and the storage saved New screenshots are picked up with a MediaStore ContentObserver and each cleanup is a unique WorkManager job keyed by the media id. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
jawzf
force-pushed
the
claude/screenshots-page
branch
from
October 15, 2026 18:41
8b824a5 to
44caae0
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested by Joe · project thread
Description
Follow-up to #1380 (merged).
Before: screenshot cleanup was a single "Delete shared screenshots" toggle that trashed a screenshot right after you shared it.
After: a dedicated Screenshots page under Security with:
How: a MediaStore
ContentObserverin the accessibility service picks up newly saved screenshots (only ones added in the last minute, with the last seen id stored so restarts don't re-notify). Each one gets a unique per-screenshot WorkManager job; Keep cancels it, and Trash now runs it immediately. Trashing still uses theIS_TRASHEDupdate from #1380, so screenshots stay restorable for 30 days, and starred ones are skipped by the automatic cleanup. No new permissions.Type of change
UI changes
The new Screenshots page.
The auto-clean notification with Keep and Trash now.
The message shown when a shared screenshot is scheduled for later.
Several screenshot notifications grouped together.
Checklist
Before submitting, please make sure:
develop(notmain)../gradlew assembleDebug../gradlew testDebugUnitTest.SettingsRepositoryfor keys and getter/setter methods.FeatureRegistry.ktfor search indexing.ui/core/(RoundedCardContainer,IconToggleItem, etc.).res/values/strings.xmland drawables use rounded variants (R.drawable.rounded_*).🤖 Generated with Claude Code
https://claude.ai/code/session_01CVTvKLdph38zfD5D45PjJh