-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Enhance Keybinding Display #3706
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
base: master
Are you sure you want to change the base?
Conversation
|
Specifying key combinations rather than actions feels backwards – the user cares about how to do an action (eg. Quit = Ctrl-q), not necessarily what a specific key combination is bound to (Ctrl-q = Quit). (Although I realize doing it that way would be much trickier to implement.) |
…ers (&, |, ,) to avoid matching similarily named actions
Yep, currently
We should add a validator for this option, which checks for bindable actions, right? Edit: |
Dynamically display keybindings using a configurable string of actions:

Might be a good idea to also document this in the help page somewhere