Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

vim vg$ only moved the selection, it does not select #8

Closed
sandy081 opened this issue Jul 27, 2016 · 4 comments
Closed

vim vg$ only moved the selection, it does not select #8

sandy081 opened this issue Jul 27, 2016 · 4 comments
Assignees

Comments

@sandy081
Copy link
Member

From @Tyriar on July 26, 2016 17:31

  • VSCode Version: Code - Insiders 1.4.0-insider (fc167f86c1aee27951b7d4b4db51229f62d0a0ad, 2016-07-26T05:12:44.554Z)
  • OS Version: Linux x64 4.4.0-28-generic, Ubuntu 16.04
    #9699

Steps to Reproduce:

  1. Open a file
  2. Go to the middle of a line with content
  3. Type "vg$", notice the cursor is only moved, no selection

Copied from original issue: microsoft/vscode#9803

@sandy081
Copy link
Member Author

From @rebornix on July 27, 2016 6:45

@sandy081 I can repro this issue. Previously when I tried this comamnd with inSelectionMode set as true, Code will help select the line or chars but now it didn't work. There is no code change in Vim side so can you help confirm is it a regression or maybe API signagure update?

@sandy081
Copy link
Member Author

From @rebornix on July 27, 2016 6:52

We have similar selection issue with vg0 (wrappedLineStart) and vg^ (nonChar) as well.

@sandy081
Copy link
Member Author

As mentioned in #9732, this is an implementation issue in Vim sample extension. In Vim, in normal or visual mode, we always adjust the end of the line position to the one before that. So during selection, Vim sample extension is adjusting just the position instead of the complete selection.

During investigation, I also figured out that #9802 is a different issue than this.
vg0, vg^, vg$ are excluding the last character.

@sandy081
Copy link
Member Author

This issue was moved to microsoft/vscode-extension-samples#2

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants