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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[馃毇 Bug] moveCursor fails on file with tabs #931

Open
3 tasks done
DavyLandman opened this issue Sep 8, 2023 · 0 comments 路 May be fixed by #1007
Open
3 tasks done

[馃毇 Bug] moveCursor fails on file with tabs #931

DavyLandman opened this issue Sep 8, 2023 · 0 comments 路 May be fixed by #1007
Assignees
Labels
bug Something isn't working
Milestone

Comments

@DavyLandman
Copy link
Contributor

DavyLandman commented Sep 8, 2023

If you use moveCursor in a file that has tabs, recent version of vscode jump to somewhere unexpectingly. So you ask to jump to line 4 column 20, but because there are 2 tabs at the front, it actuall jumps to line 4 column 25. The loop at the end of moveCursor therefor never terminates.

Steps to Reproduce

  1. have a file that contains tabs
  2. open the file using openEditor
  3. new TextEditor().moveCursor(<line>, <column after tab>);
  4. error: TimeoutError: Unable to set cursor at position 15:15
  5. if I replace the tabs in the file with 4 spaces, it works again

Expected behavior

  • jump to location and continue
  • note, this seems to be a new bug, older version of vs code worked fine with these tests.

Environment information

  • OS:
    • Windows
    • Linux
    • macOS
  • VSCode version: 1.82.0
  • ExTester version: 5.9.2
  • Node (npm) version: 8.15
@DavyLandman DavyLandman added bug Something isn't working new-issue New issue which was not discussed yet labels Sep 8, 2023
@djelinek djelinek removed the new-issue New issue which was not discussed yet label Oct 11, 2023
@djelinek djelinek added this to the NEXT milestone Oct 11, 2023
@djelinek djelinek changed the title [Bug] moveCursor fails on file with tabs [馃毇 Bug] moveCursor fails on file with tabs Oct 17, 2023
@djelinek djelinek self-assigned this Nov 21, 2023
@djelinek djelinek linked a pull request Apr 11, 2024 that will close this issue
9 tasks
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
Status: In Review
Development

Successfully merging a pull request may close this issue.

2 participants