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

set selection=exclusive #8464

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sfinktah
Copy link

@sfinktah sfinktah commented Jun 8, 2023

In answer to feature request #2747

What this PR does / why we need it: Because visual selection in vim is retarded without set selection=exclusive.

Special notes for your reviewer: This PR only partially fixes the issue. It correctly alters the start of the selection as per traditional vim in selection=exclusive mode and appears to alter the end of the selection also, but that appearance is deceptive. Can someone with more experience offer an opinion?

TL;DR:
stop = stop.getLeft(); change the visual appearance of the selection, but not the actual selection.

postnote: this has all been implemented as "vim.selectionExclusive": true, rather than the true vim selection=inclusive|exclusive|old. Would be happy to rectify that, if/when above issue is resolved.

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