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

Moving text cursor cause an incorrect position in Chinese #22717

Closed
ikatyang opened this issue Mar 16, 2017 · 6 comments
Closed

Moving text cursor cause an incorrect position in Chinese #22717

ikatyang opened this issue Mar 16, 2017 · 6 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug editor-core Editor basic functionality verified Verification succeeded
Milestone

Comments

@ikatyang
Copy link

ikatyang commented Mar 16, 2017

  • VSCode Version: 1.10.2
  • OS Version: win7 & win10

Steps to Reproduce:

  1. Press up arrow key ⬆️
  2. Press down arrow key ⬇️

vscode-text-cursor

Chinese characters are actually 2-char width in monospace-font.

Test font: 文泉驛等寬正黑 ( download: wqy-zenhei.zip, this is an open-source font )

Test config settings.json

{
  "editor.fontFamily": "文泉驛等寬正黑"
}

Test case

一二三四五六七八九十
12345678901234567890
一二三四五六七八九十
12345678901234567890
一二三四五六七八九十
12345678901234567890
一二三四五六七八九十
12345678901234567890
一二三四五六七八九十
12345678901234567890
一二三四五六七八九十
12345678901234567890
一二三四五六七八九十
12345678901234567890
一二三四五六七八九十
12345678901234567890
一二三四五六七八九十
12345678901234567890
@alexdima alexdima added bug Issue identified by VS Code Team member as probable bug editor-core Editor basic functionality labels Mar 28, 2017
@alexdima alexdima added this to the Backlog milestone Mar 28, 2017
@alexdima
Copy link
Member

alexdima commented Mar 28, 2017

Related - #22832 .. visibleColumnFromColumn should be made wide chars aware.

@wangweixuan
Copy link

Related #9989.

@alexdima
Copy link
Member

alexdima commented Mar 1, 2018

To the verifier:

@alexdima alexdima modified the milestones: Backlog, February 2018 Mar 1, 2018
@alexdima
Copy link
Member

alexdima commented Mar 1, 2018

Fixed by PR #44259

@octref
Copy link
Contributor

octref commented Mar 2, 2018

@alexandrudima #44878

@octref octref reopened this Mar 2, 2018
@alexdima
Copy link
Member

alexdima commented Mar 2, 2018

@octref Thank you for verifying, but I believe you have attempted to verify in a build that did not contain the fix (i.e. the behaviour you describe sounds a lot like what is on stable):

kapture 2018-03-02 at 9 32 29

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug editor-core Editor basic functionality verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants