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 Option to Control keys #291

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

scotch83
Copy link

@scotch83 scotch83 commented Feb 5, 2024

This development adds the option key to the keys to be displayed when only control keys are selected in the preferences

@akitchen
Copy link
Member

akitchen commented Feb 5, 2024

Thank you for opening a PR, but the option to include the option key in the set of control keys needs to also be a user-facing option for the default visualizer, as option is not a control key, it is a modifier key.

@akitchen
Copy link
Member

akitchen commented Feb 5, 2024

A more ideal solution to this feature request is to have a progression from control keys only, to all modified keys (i.e. shift and option), then all keystrokes. This warrants some thought in the preferences UI as well.

@scotch83
Copy link
Author

Maybe we can add an option in the preferences or change the label to "only modifier keys"?

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.

None yet

2 participants