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

Support for the preview Jupyter Notebook editor #5238

Closed
raghavmallampalli opened this issue Sep 24, 2020 · 5 comments
Closed

Support for the preview Jupyter Notebook editor #5238

raghavmallampalli opened this issue Sep 24, 2020 · 5 comments

Comments

@raghavmallampalli
Copy link

Describe the bug
The preview Jupyter Notebook editor now has extension support. VSCVim works out of the box. HOWEVER there are some issues I've encountered, such as Escape not allowing the user to browse the cells because of conflict with VSCVim shortcuts. I've started this thread so that people who wish to use VSCVim with the preview editor can report their issues so that the devs can get started on bug fixes or bypasses.

To Reproduce
Hit Escape while in normal mode of the preview editor and try to browse cells.

Expected behavior
Escape in insert mode goes to normal mode, Escape in normal mode allows exiting the cell editing environment into the cell browsing environment

Environment (please complete the following information):

  • Extension (VsCodeVim) version: v1.17.0
  • Version: 1.50.0-insider (user setup)
  • OS: Windows_NT x64 10.0.19041
@coroa
Copy link

coroa commented Oct 31, 2020

FWIW, you can configure it to behave correctly (ie. first Escape, leaves INSERT, second Escape leaves editor cell) by extending your keybindings.json (Command-Shift-P -> Open Keyboard Shortcuts (JSON)) to include

[

{
  "key": "escape",
  "command": "notebook.cell.quitEdit",
  "when": "inputFocus && notebookEditorFocused && !editorHasSelection && !editorHoverVisible && vim.active && vim.mode == 'Normal'"
},

]

Don't know, where to propose/include such a keybinding to have it shipped by VsCodeVim of the shelf.

@J-Fields
Copy link
Member

J-Fields commented Dec 9, 2020

I guess this is still only available in VSCode Insiders, unless I'm missing something?

@rebornix
Copy link
Member

The notebook feature is the core part of VS Code so Vim extension would need to support that gracefully. I think it makes sense to have this keybinding in Vim. I'll make the change.

@ghost
Copy link

ghost commented Jan 29, 2021

I guess this is still only available in VSCode Insiders, unless I'm missing something?

Yes it is only in the VS Code Insider. @coroa custom settings addition seems to be the only way. I just tried it with versus without.

@kamenbliznashki
Copy link

This appears to be an issue again after working for a little bit.

vscode insiders v 1.53 / 8490d3
vscodevim v1.18.8
jupyter v2020.12.414227025

@rebornix rebornix reopened this Feb 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants