Skip to content
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

fixes #8576 Cursor position is broken at v1.26.0 #8

Closed
wants to merge 1 commit into from
Closed

fixes #8576 Cursor position is broken at v1.26.0 #8

wants to merge 1 commit into from

Conversation

HenryTSZ
Copy link

When the cursor moves in insert, there is no need to execute selection logic

What this PR does / why we need it:

This PR Fix #7159 bad handling of auto-surrounding selections by Elliot-Roberts · Pull Request #8533 · VSCodeVim/Vim introduces another new bug when solving a bug

For specific bug, please refer to Cursor position with Foldfix enabled is broken as of v1.26.0 update · Issue #8576 · VSCodeVim/Vim

I located it here through debugging and also compared it with the previous version (without this bug)

Discovered that it was this code that caused it

Which issue(s) this PR fixes

Cursor position with Foldfix enabled is broken as of v1.26.0 update · Issue #8576 · VSCodeVim/Vim

Special notes for your reviewer:

When the cursor moves in insert, there is no need to execute selection logic
@HenryTSZ HenryTSZ closed this by deleting the head repository Nov 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant