-
-
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
Insert-Normal Mode moves cursor by one #1293
Comments
Ooo good one, this makes c-o pretty weird to use! I will try to get to this tonight, should be easy |
Also note of the edge case of the first char on the line...this currently acts correctly for the first character |
I just checked out the code myself and realized, that the bug is much more general. It is caused whenever i leave the insert mode with Esc. Since ctrl-o will call |
That isn't a bug though, that is the intended behavior of Esc when in insert, right? |
Oh... I see. I just checked in vim 8.0. So i would love to fix the bug (already got it working locally), but i want to keep your style of coding.
|
Ah i see. I can just pass in another parameter :) |
@xlaech Yes of course! just tag this issue in your commit message with "fixes #1293" I think it is even easier though. Can't we just do this only for Ctrl+o new CommandEscInsertMode().exec(position.getRight(), vimState); instead of new CommandEscInsertMode().exec(position, vimState); as long is it is not the first character? |
Wonderful, @xlaech! Always glad to see more people contributing. You can also feel free to ping me on our slack channel for faster responses. I'm on vacation currently, but I'll try my best to help when I can! |
Ok. thank you. |
What did you do?
In Insert Mode i pressed
<C-o>
to go to Insert normal Mode.The Commands i use in there work, but when i come back to normal mode, the cursor has moved one character to the left.
What did you expect to happen?
I expected the cursor to stay at the old position (checked the behavior in vim 8.0)
What happened instead?
The Cursor moved one character to the left.
Technical details:
The text was updated successfully, but these errors were encountered: