Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add screenshots to housekeeping #55

Closed
danirabbit opened this issue Dec 1, 2022 · 3 comments · Fixed by #56
Closed

Add screenshots to housekeeping #55

danirabbit opened this issue Dec 1, 2022 · 3 comments · Fixed by #56
Labels
Good First Issue Small, self-contained issue. Good for newcomers, and/or should be an easy fix. Needs Design Waiting for input from the UX team

Comments

@danirabbit
Copy link
Member

Problem

I always end up taking a ton of screenshots and I just have like hundreds of files that I definitely don't need anymore

Proposal

Would be great to add the screenshots directory to housekeeping

Prior Art (Optional)

No response

@danirabbit danirabbit added Needs Design Waiting for input from the UX team Good First Issue Small, self-contained issue. Good for newcomers, and/or should be an easy fix. labels Dec 1, 2022
@Antolius
Copy link
Member

Antolius commented Dec 10, 2022

Would this behavior require a new toggle in the Housekeeping view of the System Settings, or would it be enabled by one of existing options (old temporary files perhaps)?

@danirabbit
Copy link
Member Author

@Antolius Yeah I think it should be a new checkbox since it's a different location

@Antolius
Copy link
Member

Cool! I'd like to give implementing this a try.

Btw, @danirabbit would you mind me looking into #47 as well? (If you are OK with it in principal I can open a separate PR for that.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good First Issue Small, self-contained issue. Good for newcomers, and/or should be an easy fix. Needs Design Waiting for input from the UX team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants