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

Make escape not close stonesense, or make it confirm/overrideable #87

Open
robob27 opened this issue Dec 21, 2022 · 1 comment
Open

Comments

@robob27
Copy link

robob27 commented Dec 21, 2022

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?

@robob27
Copy link
Author

robob27 commented Dec 21, 2022

Resolved this locally by removing

stonesense/main.cpp

Lines 362 to 363 in 2908625

} else if (event.keyboard.keycode == ALLEGRO_KEY_ESCAPE) {
return;

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

No branches or pull requests

1 participant