-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Move commandline to status bar to allow history navigation #2758
Conversation
The only thing I can't figure out is how to successfully use to paste append to the text, works fine, but we special cased |
Travis tests have failedHey @xconverge, Node.js: 8npm test --silent
|
Travis tests have failedHey @xconverge, Node.js: 8npm test --silent
|
Hey @xconverge, TravisCI finished with status |
@jpoon can you take a quick pass on this when you get a chance? Functionally everything is great now, but I am open to feedback! Any thoughts on me removing the showCmdLine command, I don't think anyone will mind that but I could be wrong... Tested |
This is awesome. Thanks for doing this.
Fine by me! I don't see why somebody would use it either. Quicker to type |
Thanks for the awesome work! |
I did use it, to enable a mapping that I use frequently in native Vim, My old config was: "vim.normalModeKeyBindingsNonRecursive": [
// <Enter> => \n
// https://github.com/VSCodeVim/Vim/issues/1895
{
"before": ["\n"],
"commands": [{ "command": "extension.showCmdLine" }]
}
] I've updated it with |
Fixes #779
This is our most thumbs upped/sad faced issue currently!
I did not do any complex or majestic refactoring. I mostly took what worked well with searching and applied it to the commandline. Works pretty well!