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

Added option Empty Trash #1558

Merged
merged 2 commits into from
Feb 24, 2018
Merged

Added option Empty Trash #1558

merged 2 commits into from
Feb 24, 2018

Conversation

pfftdammitchris
Copy link
Contributor

@pfftdammitchris pfftdammitchris commented Feb 19, 2018

Feature request from #1555

I was contemplating to add a window to allow the user to "confirm" the deletion, but I think that wouldn't be needed since the user already technically "confirmed" that decision when deleting a single note beforehand. Apps nowadays don't really give the user any notification of emptying the trash anyways because 99% of the time the user never really empties the trash on accident. nor would the notes being deleted from the trash be of importance anyway, being the reason they got there in the first place

emptytrash

@Rokt33r Rokt33r self-requested a review February 20, 2018 07:33
@Rokt33r Rokt33r added the awaiting review ❇️ Pull request is awaiting a review. label Feb 20, 2018
@Rokt33r Rokt33r added next release (v0.9.1) and removed awaiting review ❇️ Pull request is awaiting a review. labels Feb 24, 2018
@Rokt33r Rokt33r merged commit 2750a3e into BoostIO:master Feb 24, 2018
@pfftdammitchris
Copy link
Contributor Author

pfftdammitchris commented Feb 24, 2018

Close #1555

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.

None yet

3 participants