-
Notifications
You must be signed in to change notification settings - Fork 674
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
vi mode? #3
Comments
Added a pull request |
…xceptions Fix first-chance exceptions in Windows A call to WriteConsoleOutputCharacter() didn't have its final parameter set, passing a zero instead of a pointer to where to return the char count. I added the same code that was there for FillConsoleOutputCharacter() and FillConsoleOutputAttribute(), just allocate a dummy 'n' and pass its address.
…tion Clarify the comment at the definition of IGNORE_RC.
The best feature would be is to have vim mappings. Then I can throw away my .inputrc and start using some imap and nmap shortcuts for all my abbreviated commands. Bash with normal mode would make the command line as productive as the editor. :)
|
+1 Thank you @bobrippling for your patch - it adds support basically for the whole vi mode, which is awesome! Actually those 150 new lines are imho definitely worth including this patch into this original repository. |
+1 This would be a fantastic improvement from my point of view, as I'm using another project (Planck) that uses linenoise. Is anything blocking the pull request from being merged? Thanks! |
This issue is addressed by #92. |
Looks like #92 just got close without merging 😞 |
This would seriously up the line count, but I'm also an addict that can't get enough of vi/vim.
Some knowledge of modes, with support for Esc, i, h, l, j & k are the minimal requirements.
The text was updated successfully, but these errors were encountered: