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

IME: handle missing compositionend events for Sogou IME #3680

Merged
merged 2 commits into from
Mar 17, 2022

Conversation

Eugeny
Copy link
Member

@Eugeny Eugeny commented Mar 12, 2022

Fixes #3679

This relies on checking for a missing keydown event before deciding whether to ignore input(insertText) events - see #3679 for specific details.

All this gives me the jibbies, so tested with:

  • macOS / US / alt-U umlauts
  • macOS / Swedish / alt-" umlauts
  • macOS / Emoji
  • Windows / Sogou in Chrome
  • Windows / Sogou in Electron
  • Windows / Emoji
  • Linux / Sogou in Chrome
  • Linux / Sogou in Electron
  • Linux / Compose key

@Eugeny Eugeny changed the title ime: handle missing compositionend events for Sogou IME IME: handle missing compositionend events for Sogou IME Mar 14, 2022
@Tyriar Tyriar added this to the 4.19.0 milestone Mar 17, 2022
@Tyriar
Copy link
Member

Tyriar commented Mar 17, 2022

Thanks @Eugeny, makes sense. Fingers crossed we can close off this chapter 🙂

@Tyriar Tyriar merged commit eb562b5 into xtermjs:master Mar 17, 2022
@Eugeny Eugeny deleted the electron-13-sogou-fix branch March 17, 2022 16:34
@hsj1606
Copy link

hsj1606 commented May 1, 2022

Hello @Eugeny, with Sogou IME english mode on macOS, if you type relatively fast, some characters will be lost

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

Successfully merging this pull request may close these issues.

CJK IME (Sogou) input broken on Windows in Electron 13
3 participants