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

feat: add an option to disable moving the discarded changes to Trash #17436

Open
wants to merge 3 commits into
base: development
Choose a base branch
from

Conversation

aminya
Copy link
Contributor

@aminya aminya commented Sep 24, 2023

Closes #10445
Remedy for #7155

Description

This adds an option to disable moving the discarded changes to Trash, i.e. permanently discard.

Screenshots

image
image
image

Release notes

Notes:

  • feat: add an option to disable moving the discarded changes to Trash

@aminya aminya changed the title feat: add an option to permanently discard changes feat: add an option to disable moving discard changes to Trash Sep 25, 2023
@aminya aminya changed the title feat: add an option to disable moving discard changes to Trash feat: add an option to disable moving the discarded changes to Trash Sep 25, 2023
@MichaIng
Copy link

Many thanks, such an important feature.

  • The "Do not show this message again" checkbox refers to the discard confirmation prompt itself, not the trash bin choice, right?
  • If so, if you choose to not show the confirmation prompt again, you cannot uncheck the trash bin usage? If so, is the trash bin choice saved for all future files/discards?
  • I have not tested in a while, but discarding multiple files seems to always enforce a confirmation, hence no "do not show again" checkbox?
  • Finally, I think it would be great to have the checkbox in the regular options GUI as well, to not solely rely on the confirmation dialog. But that should not be a blocker for this PR.

@aminya
Copy link
Contributor Author

aminya commented Sep 29, 2023

You're welcome!

The "Do not show this message again" checkbox refers to the discard confirmation prompt itself, not the trash bin choice, right?
If so, if you choose to not show the confirmation prompt again, you cannot uncheck the trash bin usage? If so, is the trash bin choice saved for all future files/discards?
I have not tested in a while, but discarding multiple files seems to always enforce a confirmation, hence no "do not show again" checkbox?

I didn't touch the code related to "do not show". Whatever logic you have there should work as expected independently.

Finally, I think it would be great to have the checkbox in the regular options GUI as well, to not solely rely on the confirmation dialog. But that should not be a blocker for this PR.

I added that in bdd16a6

@aminya
Copy link
Contributor Author

aminya commented Jan 16, 2024

@niik Any update on merging this?

@lofcz
Copy link

lofcz commented Jan 27, 2024

Is there something blocking the merge? Just had to discard 15k files, github desktop client didn't even finish after hours, git reset HEAD && git clean -fd took ~10 seconds.

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.

Add option to not use Trash bin
3 participants