-
-
Notifications
You must be signed in to change notification settings - Fork 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
Vim mode #103
Comments
@clouedoc I would definitely consider researching this! Also looking forward for contributions from community. Afaik there is a VScode plugin for vim mode. Probably its a good place to start researching. |
https://github.com/brijeshb42/monaco-vim - thoughts on testing this out and including it as a package dependency? Will test out locally |
@fow5040 I'll take a look, thanks |
FYI - I did test it out and it seems to play nicely/not conflict with the go-playground instance of monaco editor. Obviously there's a cleaner way to do this but it seems to work just fine: this.vimMode = initVimMode(
this.editorInstance,
document.getElementById("vimStatus"),
); I'll hold off on any further development - thanks a ton for looking into this! |
@fow5040 looks like I finished it. Had to add a lot of glue code on top of lib's glue code (because this lib is basically a big adapter on top of another lib) so I'm not 100% sure about stability. I have other few things to do before shipping a new release, but it will be shipped in v1.11 soon. Can I ask you to try to test the latest |
Gladly, thanks a ton! |
@x1unix Minor:
Nitpick
Lastly, I managed to abuse the vim plugin for a while and somehow got my editor into a "call stack exceeded" state where the whole editor stopped responding. Unclear if this was because of the vim plugin or some other sort of abuse I did, but I haven't been able to reproduce. Probably a symptom of gluing, and not a huge issue As a disclaimer, while I work in vim most of the day, I am by no means a vim guru. Here are a handful of things I did test that seem to be working fine:
All in all, considering this is a go playground and not necessarily a full fledged IDE, I think the state of the vim plugin is entirely usable in its current state (especially considering it's not enabled by default). |
@fow5040 thank you for feedback. I'll add a fix for search and registry commands. Unfortunately currently I can just implement a Maybe later I'll implement some better solution. |
Hi, your Golang playground is nice!
Would you consider adding support for Vim keybindings as an option to add in the settings menu?
The text was updated successfully, but these errors were encountered: