-
Notifications
You must be signed in to change notification settings - Fork 73
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
Ctrl-[ and Ctrl-C to exit insert mode #85
Comments
Interesting I didn't realize ctrl-[ wasn't working on all platforms, I typically only use chrome. I just tried Firefox and it seems like it was working for me? I check Safari and it was working there for me as well. I'm using Firefox 68.0 (Quantum) and jupyterlab-vim 0.11.0. |
Maybe it is a plugin that is capturing the key, but when I use CTRL-[, it leaves the cell instead of going to normal mode (which is annoying since I have to re-click on the cell to have the focus again). However, if I add the keybinding in Using Firefox 68.0.1, jupyterlab 1.0.2, jupyterlab-vim 0.11.0 @llinfeng seems to have the same issue since he said in #81 that he recompiled the plugin with CTRL-[ added in the keybindings. |
@axelfahy Were you also recompiling the plugin when you "added the keybindings in
I have compared things side-by-side across Chrome, Edge and Firefox
Raw conclusion: something is wrong with Firefox :) May need further debugging to see what captures |
Yes, I recompiled the plugin after adding the keybinding. Afterwards, CTRL-[ worked fine. |
In vim,
Ctrl-[
is the equivalent of Esc andCtrl-C
quits insert mode. I think it would be nice to have the same behaviors in jupyterlab-vim.Maybe implementing
Ctrl-C
is not a good idea since it will interfere with the copy shortcut. However, I think thatCtrl-[
won't interfere with other shortcuts.Reference from vim documentation (https://vimhelp.org/insert.txt.html):
CTRL-[
CTRL-C
Update:
Ctrl-[
seems to be working on Chrome but not on Firefox (v67.0.4), not sure why. (other browsers not tested)The text was updated successfully, but these errors were encountered: