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

Add insertFromComposition and deleteByComposition inputTypes #12125

Closed
wants to merge 1 commit into from

Conversation

ZhangYiJiang
Copy link
Contributor

@ZhangYiJiang ZhangYiJiang commented Aug 22, 2021

Summary

Add Input Events Level 2 insertFromComposition and deleteByComposition inputType events

Test results and supporting details

For each browser, on macOS pinyin IME input a 1 (inserting 啊), then selecting the inserted text and input b 1 (deleting 啊 and replacing it with 不). According to https://w3c.github.io/input-events/#event-order-during-ime-composition we should see

  • deleteByComposition beforeInput and input events for the second test since 啊 is being deleted
  • insertFromComposition beforeInput and input events for both cases

Firefox: https://input-inspector.vercel.app/profiles/8n74DRvc3g5dhl6pMYQo - neither event fires
Chrome: https://input-inspector.vercel.app/profiles/AXWWGKNTootDNMo6pXcY - neither event fires
Safari: https://input-inspector.vercel.app/profiles/Hocw2xtbGkIbqgFwXzQP - insertFromComposition fires but deleteByComposition does not. I don't know when this was added, so this is just left as true

Firefox - the original bug for supporting inputEvent notes both these level 2 events will not be implemented yet https://bugzilla.mozilla.org/show_bug.cgi?id=1447239

Related issues

@github-actions github-actions bot added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Aug 22, 2021
@queengooborg queengooborg added the not ready ⛔ This is not yet ready to be merged. It's pending a decision, other PR, or a prerequisite action. label Aug 23, 2021
@queengooborg
Copy link
Collaborator

Thank you for your PR, and welcome to BCD! Currently, we are in the process of deciding whether or not to allow all-false entries in BCD -- see #10619 for more details. I will be marking this as "not ready" accordingly.

@queengooborg
Copy link
Collaborator

Hey again @ZhangYiJiang, thank you for your patience on this! We have decided not to include features that have no compatibility in BCD. A spec may implement a feature, but then rename or remove it before any browsers implement it, thus leaving a leftover statement in BCD.

Would you be down to research if any browsers support this feature now? If not, I'd be happy to look into it and follow up!

@queengooborg
Copy link
Collaborator

I'm going to close this now since it's been over two months and there has been no response and I'm seeing no signals that any browser supports these. Thank you anyways!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API not ready ⛔ This is not yet ready to be merged. It's pending a decision, other PR, or a prerequisite action.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants