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

undo with u does not work on jupyterlab 4 #89

Closed
navdeeprana opened this issue Jul 31, 2023 · 8 comments · Fixed by #92
Closed

undo with u does not work on jupyterlab 4 #89

navdeeprana opened this issue Jul 31, 2023 · 8 comments · Fixed by #92
Labels
bug Something isn't working

Comments

@navdeeprana
Copy link

Thank you so much for maintaining and porting the extension to jupyterlab 4. I upgraded to the latest versions of both and undo with u does not work. Ctrl-Z does.

@navdeeprana navdeeprana added the bug Something isn't working label Jul 31, 2023
@ianhi ianhi pinned this issue Jul 31, 2023
@lukashergt
Copy link

lukashergt commented Jul 31, 2023

I can confirm, u within a cell does not work.

In "jupyter" mode, you can create a new cell with o and undo that with u.

@ianhi
Copy link
Collaborator

ianhi commented Jul 31, 2023

Thanks for reporting! I can confirm that I experience this. I wonder if this is the same cause as #31 where we were colliding with a default jupyter shortcut somehow.

It's not because we switched to the new codemirror vim as it's works here: https://replit.com/@util/codemirror-vim

@ianhi
Copy link
Collaborator

ianhi commented Jul 31, 2023

maybe related to jupyterlab/jupyterlab#14113

@ianhi
Copy link
Collaborator

ianhi commented Jul 31, 2023

digging deeper still it seems that undo has become a more complicated action in jupyterlab 4. probably to help with the real time collab stuff. It seems as though ctrl-Z isn't handled by codemirror, but rather by jupyterlab. in particular I think it ends up calling this funciton: https://github.com/jupyterlab/jupyterlab/blob/659c6113a97cb906c8b90867a1fe916e628547f2/packages/codemirror/src/editor.ts#L263-L265

so we'll need to end up overriding what u does and have it call that funciton somehow

@krassowski
Copy link
Collaborator

That's doable. I can open a PR on top of #90.

@krassowski krassowski mentioned this issue Jul 31, 2023
@krassowski
Copy link
Collaborator

PR is up: #92

@ianhi ianhi closed this as completed in #92 Jul 31, 2023
@ianhi
Copy link
Collaborator

ianhi commented Jul 31, 2023

fix is now released.

pip install --upgrade jupyterlab-vim

@navdeeprana
Copy link
Author

thank you so much for the quick fix

@ianhi ianhi unpinned this issue Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants