-
Notifications
You must be signed in to change notification settings - Fork 85
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
Right click to close main menu or exit current tool #1032
Conversation
…l - set current tool to None, update OSD
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good stuff
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the code LGTM
Good point, it's driven by separate logic (cool spaghetti), I'll fix it 🙃 |
I've implemented fix for closing road selection panel on single right click, @chameleon-tbn your turn and I'll merge if you won't find anything else. Good addition to Stable 11.5 |
I'll test today in full modded game :) |
I've tested now in a game with 135 Mods, 10k assets. All seems to work as expected - even if i really need to 'relearn' using the right click in the correct way :) 1x = closing actual activated UI, 2x = leave the actual selected button, 3x = completely close TM:PE UI... really handy... i like :) a go from me :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Closes #1014
Compiled version for tests
New:
Improvements for existing code: