-
Notifications
You must be signed in to change notification settings - Fork 16
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
Partial committing composition definition is more complicated than actual implementations #87
Comments
Isn't that the same thing point 6 says? |
No, I meant that should restart from point 2, i.e., with |
Ah I see. This was actually the case at one stage - one compositionstart could have several compositionend events. Isn't this issue a duplicate of w3c/uievents#202 ? |
No, it's issue about event order between |
Ok, but it's closely related. The thing about multiple |
6. . Event order during IME composition defines partial committing composition case at 6 and the following NOTE.
However, as far as I've tested, Firefox, Chrome and Safari does not handle partial commit as such. All of them commits (long) composition with only partial committed string. Then, restart composition with
compositionstart
event andcompositionupdate
event with remaining composing string.I believe that current browser implementation makes more sense because web apps do not need to support partial commit by themselves and web apps can now composing range only with retrieving selected range at
compositionstart
.So, there is no problem with defining
compositionend
andcompositionstart
are always pair.I tested with "live conversion mode" of Apple Japanese Input and ATOK on macOS (but IIRC, on Windows, same result).
(And I guess that IME vendors need to implement as so for compatibility with all existing IME-aware native applications. Windows API has partially committed composition string representation with treating committed part as "fixed" clauses. However, I've never meet such IME even though I work on maintaining IME handler of Gecko for more a decade.)
The text was updated successfully, but these errors were encountered: