-
Notifications
You must be signed in to change notification settings - Fork 55
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
Make escape not close stonesense, or make it confirm/overrideable #87
Comments
Resolved this locally by removing Lines 362 to 363 in 2908625
|
I, as an end user, do like the esc to close feature, but perhaps we could make a "CONFIRM_ON_CLOSE" config to make it double-check if you wanna close it. |
changing to have a edit: added as part of PR #121 |
realSquidCoder
added a commit
to SquidCoderIndustries/stonesense
that referenced
this issue
Jan 13, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I have closed the stonesense window with the escape key many many times by accident, but never on purpose. It seems unnecessary with the other options available to close the app (X button, ALT+F4) unless I'm missing something. If there is a good reason for it, perhaps we could add it to the keybinds that are changeable, or at the very least give it a confirmation dialog?
The text was updated successfully, but these errors were encountered: