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

EditContext spec should define composition events as they are (not?) fired on EditContext editing host #72

Open
masayuki-nakano opened this issue Oct 13, 2023 · 2 comments
Labels
needs-spec-edits Issue was resolved but spec needs to be updated

Comments

@masayuki-nakano
Copy link

https://w3c.github.io/edit-context/#edit-context-differences

The EditContext editing host receives key events and the beforeinput event as specified in [uievents].

IIRC, EditContext has its own way to notify web apps of composition. So I think composition events should not be fired on there, but it's not defined in the "Differences for an EditContext editing host" section.

@johanneswilm
Copy link

johanneswilm commented Oct 13, 2023

@masayuki-nakano Does that not go against what we resolved on in #69 where we decided that the JS author should listen to composition events?

@johanneswilm johanneswilm transferred this issue from w3c/editing Oct 13, 2023
@dandclark
Copy link
Contributor

In #69 we removed some properties from the EditContext but I don't think it affected when and where events fire.

I agree with @masayuki-nakano that it looks like https://w3c.github.io/edit-context/#edit-context-differences should reflect that composition events are not fired on the EditContext editing host, since they are fired on the EditContext instead.

This is not a change in behavior, just something that is unclear in the spec.

@dandclark dandclark added the needs-spec-edits Issue was resolved but spec needs to be updated label Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-spec-edits Issue was resolved but spec needs to be updated
Projects
None yet
Development

No branches or pull requests

3 participants