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

Undo/Redo adds a stop in between CHN Characters #9675

Closed
yangjianhua opened this issue Jul 25, 2016 · 6 comments
Closed

Undo/Redo adds a stop in between CHN Characters #9675

yangjianhua opened this issue Jul 25, 2016 · 6 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug editor-core Editor basic functionality l10n-platform Localization platform issues (not wrong translations) verified Verification succeeded
Milestone

Comments

@yangjianhua
Copy link

  • VSCode Version: Code 1.3.1 (e6b4afa, 2016-07-12T13:35:06.227Z)
  • OS Version: Windows_NT ia32 6.3.9600

Steps to Reproduce:

1.Put any Chinese characters (more than 1) in any file of vscode;
2.Press CTRL+Z to undo the action above;

Separate a single CHN character into 2, so we need to press double CTRL+Z in coding.

This is the main reason I still use Sublime Text

@lextm
Copy link

lextm commented Jul 25, 2016

I cannot reproduce this issue with Google Pinyin on Windows 7 x64 though I am still using VSCode 1.2.1 on this machine. But what I observed is also interesting,

Type "ceshi" and Google Pinyin turns it to "测试". Then Ctrl + Z rolls it back to ceshi.

This behavior exactly matches how Ctrl + Z works inside Chrome.

I don't think this is a bug. It sounds more like a limitation in Chromium IME integration. Note that VSCode is in fact a browser, not a classic desktop app like Sublime Text.

@yangjianhua
Copy link
Author

yangjianhua commented Jul 29, 2016

In fact it really exists.

I don't know what "Chromium IME integration" mean, I'm using Win 8 as the first post grab.

And it really exists.

it's still the biggest problem that I insist to using Sublime Text

@lextm
Copy link

lextm commented Jul 30, 2016

The issue can exist if you use a specific Chinese input method (you don't even mention which you are using), as I think Control + Z behavior is heavily tied to the input method itself. I just said that VSCode does nothing wrong, and Chromium based applications (such as Google Chrome) like I tested behave the same.

@dbaeumer dbaeumer removed their assignment Aug 3, 2016
@yaxinlx
Copy link

yaxinlx commented Aug 7, 2016

still exists in v1.4, Debian amd64, SCIM+Smart Pinyin.

对一个刚输入简单的中文词语,需要ctrl+z七八次才能完全清除此中文词语。
没装过vsc1.1-1.3,但是v1.0中正常的。感觉vsc创建了太多的undo points.

@alexdima
Copy link
Member

alexdima commented Aug 8, 2016

@yangjianhua @yaxinlx Can you please help me out and provide steps for dummies :) -- similar to those described in https://github.com/Microsoft/vscode/wiki/IME-Test

Then, I can try to reproduce and check what happens at the model level that would make undo confused and not stitch together multiple edits in a single undo operation.

@alexdima alexdima added the info-needed Issue requires more information from poster label Aug 8, 2016
@alexdima alexdima modified the milestones: August 2016, Backlog Aug 8, 2016
@yaxinlx
Copy link

yaxinlx commented Aug 9, 2016

@alexandrudima

Debian packages needed to use smart pinyin: apt-get install scim-pinyin fonts-arphic-ukai fonts-arphic-uming

Restart may be needed to apply the change.

Open vsc1.4, use ctrl+space to toggle input methods.

When Chinese input method is enabled, try inputing "long", pressing space to select a character. Then press crl+z to undo, you need undo 8 times to clear the just input character.

It looks vsc creates two undo points for each letter typing.

@alexdima alexdima added the bug Issue identified by VS Code Team member as probable bug label Aug 9, 2016
@alexdima alexdima changed the title Can not handle CHN Characters Undo/Redo adds a stop in between CHN Characters Aug 26, 2016
@alexdima alexdima added l10n-platform Localization platform issues (not wrong translations) and removed info-needed Issue requires more information from poster labels Aug 26, 2016
@alexdima alexdima modified the milestones: August 2016, Backlog Aug 26, 2016
@alexdima alexdima added editor editor-core Editor basic functionality labels Aug 30, 2016
@rebornix rebornix added the verified Verification succeeded label Aug 31, 2016
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
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 l10n-platform Localization platform issues (not wrong translations) verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

6 participants