-
Notifications
You must be signed in to change notification settings - Fork 22
Upgrade to CM 4.7.0+ #45
Comments
This issue is now closed. Should we look into resolving this now? |
codemirror has done lots of work on vim lately. anything i can do to help get LT updated to the latest version? |
@bjarthur Sure! Using the readme, upgrade vim.js and figure out why vim is stuck in insert mode and can't switch to normal mode. I've also noticed some keys don't work in insert mode e.g. arrow keys and ctrl on osx. There's a possibility that we'll also need to upgrade codemirror.js in order for this to work (#1797). Historically we haven't had to upgrade codemirror.js and vim.js at the same time but there's a possibility recent changes require they both be upgraded. I have higher priority issues to look at for now so any help appreciated in the meantime. |
what are the developer tools for debugging here? how do i set a breakpoint in vim.js? where does console.log() go? is there an IDE? |
@bjarthur You should be able to use DevTools – LT runs on top of node-webkit (or { Atom shell / Electron}), which is basically a browser. |
This temp fix can be removed once we upgrade CM version - #45. Removed make-normal-editor because we don't use it anywhere
4.7.0 introduced custom keypress/keydown handlers. This conflicts with LT's mousetrap handlers and causes Esc, backspace. Waiting to upgrade until codemirror/codemirror5#2915 is addressed
The text was updated successfully, but these errors were encountered: