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
Allow platform aware key shortcuts #7589
Conversation
Normalizing keys will return a list of shortcuts available for the running platform (so platform specific keys + keys). Process that normalized list instead of the shortcut keys directly. This means that winKeys, linuxKeys, and macKeys are properly supported in jupyterlab shortcuts.
Phosphor allows shortcuts to be empty, and it’s the only way to have platform-specific shortcuts (define keys to be [], and the platform keys to be something specific). See phosphorjs/phosphor#438 for some discussion.
Thanks for making a pull request to JupyterLab! To try out this branch on binder, follow this link: |
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.
I merged in master to take care of conflicts, and tested once again. Looks good to me!
The Linux usage test failure looks like it is a Lumino issue that should be solved there, and looks like it is specific to the testing framework not running in an actual browser?
Thanks @jasongrout. |
References
Supersedes #7581
Code changes
Relies on jupyterlab/lumino#11 to normalize key shortcuts before reconciling key shortcuts in jupyterlab.
Waiting on a new release of lumino before finishing this PR. I will comment below when ready for review. Thanks!
User-facing changes
Backwards-incompatible changes