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

Cursor move around #211

Closed
ciaranchen opened this issue Dec 14, 2019 · 5 comments
Closed

Cursor move around #211

ciaranchen opened this issue Dec 14, 2019 · 5 comments
Labels
bug Something isn't working
Milestone

Comments

@ciaranchen
Copy link

When I selected a text range, the anchor move around. I disable all other extensions, but the question still exists.

I thought the question is in src/mirrorCursor.ts. the window.onDidChangeTextEditorSelection was called multiple times while I selected a text range. It's parallel.

Then in event handler, it calls exitMirrorMode which change selections.

So when I selected a range of text quickly, anchor will go back to the charator I selected before. And if I click on other position, the anchor will move around the whole page.

Sorry for my poor English. I have tried.

屏幕录制2019-12-14下午12 25 38

@ciaranchen ciaranchen changed the title Anchor move around Cursor move around Dec 14, 2019
@fbricon
Copy link
Collaborator

fbricon commented Dec 14, 2019

@ciaranchen try toggling the cursor mirror mode with Ctrl+Shift+F2 (or Cmd+Shift+F2 on mac)

@NikolasKomonen @xorye when mirror mode is disabled, the event handler should be removed so as to avoid that.

@fbricon fbricon added the bug Something isn't working label Dec 14, 2019
@fbricon
Copy link
Collaborator

fbricon commented Dec 14, 2019

This is really bad
Dec-14-2019 11-05-25

@daveb-501commons
Copy link

daveb-501commons commented Dec 14, 2019

Happening for me as well - disabled the vscode-xml extension and all is working again

microsoft/vscode#86966 (comment)
eclipse-lemminx/lemminx#601

@fbricon
Copy link
Collaborator

fbricon commented Dec 15, 2019

0.10.1 released, the mirror cursor editing has been temporarily removed

@fbricon fbricon closed this as completed Dec 15, 2019
@angelozerr
Copy link
Contributor

Thanks @fbricon !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants