Skip to content

Clear input state if applicable before hopping out of normal mode to jupyter command mode #82

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

Merged
merged 1 commit into from
Aug 3, 2023

Conversation

ryantam626
Copy link
Contributor

For example if I press d then <Esc> in normal mode (i.e. about to delete something but changed my mind), I expect to be still in normal mode instead of jupyter command mode.

@github-actions
Copy link

github-actions bot commented Jun 1, 2023

Binder 👈 Launch a binder notebook on branch ryantam626/jupyterlab-vim/rt.incremental-esc-fix

@ryantam626 ryantam626 force-pushed the rt.incremental-esc-fix branch from c8051c5 to a31eee7 Compare June 1, 2023 16:27
@ianhi
Copy link
Collaborator

ianhi commented Aug 1, 2023

@ryantam626 sorry I wasn't able to look at this when it was opened. This looks quite good and important for resolving #73

do you have the time to rebase? if not I'm happy to take over this PR

…jupyter command mode

For example if I press `d` then `<Esc>` in normal mode (i.e. about to
delete something but changed my mind), I expect to be still in normal
mode instead of jupyter command mode.
@ryantam626 ryantam626 force-pushed the rt.incremental-esc-fix branch from a31eee7 to c72e82e Compare August 1, 2023 20:26
@ryantam626
Copy link
Contributor Author

@ianhi done 🙌 - looks like I am not supposed to touch the changelog so I have left it alone.

@ianhi ianhi added the enhancement New feature or request label Aug 1, 2023
@ianhi
Copy link
Collaborator

ianhi commented Aug 3, 2023

this looks correct. but just to ensure im testing it properly, how are you invoking this command? Have you modified the keyboard shortcuts to bind esc to leave-current-mode?

@ianhi
Copy link
Collaborator

ianhi commented Aug 3, 2023

ok turns out i really had to muck around with my keyboard shortcuts to get that to go. not sure why I had such trouble. but certainly seems good to me. thanks!

@ianhi ianhi merged commit 9479e09 into jupyterlab-contrib:master Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants