-
Notifications
You must be signed in to change notification settings - Fork 5k
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
[WIP] Adding classic shortcuts extension #7075
base: main
Are you sure you want to change the base?
Conversation
- Enable/Disable mechanism
- Delete line shortcut
Thanks @RRosio for looking into this! This would help some folks migrate from the classic notebook to Notebook 7. Wondering if this new extension should only be available in Notebook 7 though? Or could it be that some JupyterLab users would also like to have this set of keyboard shortcuts available? If so, maybe it should be its own extension? (for example in https://github.com/jupyterlab-contrib) |
Thank you for taking a look at this @jtpio! |
True. Maybe the main advantage would be to have a coherent set of shortcuts between JupyterLab 4 and Notebook 7.
Normally the number of clicks or other UX considerations should not depend on whether an extension lives in a different repo? Unless it requires an extra
So similar to the custom CSS feature, maybe the best would indeed be to develop the extension in the notebook repo like in this PR, and include it as part of the Notebook application but not the JupyterLab application. If some users show interest to have the extension enabled in JupyterLab as well, we could convert the extension to a regular JupyterLab extension but still develop it in the |
Adding to the |
@RRosio just checking in to see if you would like to have this for the 7.1 release? I would like to publish a first 7.1 beta release soon, to try to stay in sync with the JupyterLab release cycle. Otherwise no problem to postpone it for later and consider that for |
Hi @jtpio sorry I didn't see this message earlier! I think I would like to say this could be ready for a 7.2 release rather than 7.1, it is still not working quite like I intended and it is a bit flaky at times. I am working on it but if I fix it quickly and a 7.1 release is happening soon I don't want to hold it back. |
Thanks so much for working on this! 🙏🏻 This is an important/required feature for notebook power users (speaking for myself and several teammates of mine), so I'm glad to see a little bit of activity here. It's maybe worth mentioning that folks who have worked for years in nb "classic" don't give notebook 7 or jupyterlab more than a few seconds chance, if that, because the ui elements or what it is that nb 7 features/targets, are not worth the hit to velocity or loss of utility one has in classic. Feature parity with classic should be the acceptance criteria for updating notebook. So I appreciate your support here! I hope this gets higher priority! (Context for this comment being my team's env got bumped onto 7 by accident and we had to revert) |