-
-
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
[Fix] Don't add beginning newline of linewise put in visual-mode #2579
[Fix] Don't add beginning newline of linewise put in visual-mode #2579
Conversation
Travis tests have failedHey @tyru, Node.js: 8npm test --silent;
|
oops, okay, we should fix |
Travis tests have failedHey @tyru, Node.js: 8if [[ $(git diff-index HEAD --) ]]; then git diff; echo "Prettier Failed. Run `gulp` or `gulp forceprettier`"; exit 1; fi
|
Fixed the problem and I added the test case too. Btw, is there a way to run only certain suites/tests? |
d7fe95e
to
3db4177
Compare
It seems just I forgot to transpile test code... ;) |
@tyru To only run certain test cases, write |
@Chillee Ah, thanks! it saves a lot of time! 😄 |
This has been bugging me so much, thanks! |
Thanks all! |
Thanks for great work.
I'm using Vim for primary text editor but I have to use VSCode at work.
But I found this extension is awesome because Vim like keybinding helps me a lot to concentrate :)
What this PR does / why we need it
In original Vim behavior,
p
command in visual-mode replaces current text.VSCodeVim also does it but it is slightly different.
Unnecessary newline is inserted before the text when the text is yanked as linewise.
yy
Vp
In original Vim, it does not change the text (the buffer is marked modified, though).
But VSCodeVim inserts a newline before the current line.
Which issue(s) this PR fixes
No exact issue yet.
But the problem is touched by #1991 comment.