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: processing model #23

Closed
annevk opened this issue Dec 10, 2021 · 2 comments
Closed

EditContext: processing model #23

annevk opened this issue Dec 10, 2021 · 2 comments

Comments

@annevk
Copy link
Member

annevk commented Dec 10, 2021

The larger issue with this specification I wanted to raise is two-fold:

  1. It's very much unclear how this integrates with the larger web platform processing model. EditContext: "is activated" is not defined #20 and EditContext: "it is focusable" is underdefined #21 touch upon this. How does this change the model of elements and focusing? What user input actions end up redirected here vs the element?
  2. The processing model of the EditContext object itself is unclear. EditContext: define internal model #22 touches upon this. For instance, at some point the draft says "Update the text, selectionStart, selectionEnd, compositionRangeStart, compositionRangeEnd, and isInComposition properties of the activated EditContext." but it doesn't actually say what any of these values are supposed to be set to and what that depends upon.
@alexkeng
Copy link
Contributor

We appreciate the feedback and agree that more clarity is needed. We'll address related issues and add more details in the next revision

@travisleithead travisleithead transferred this issue from w3c/editing Apr 21, 2022
@dandclark
Copy link
Contributor

We've made a lot of changes that I think address the issues raised here. See the sub-issues linked from this issue for specific fixes.

See also w3c/input-events#145 which we're not going to land until EditContext is shipping/close to shipping. That PR defines the entry point for user input.

I'm going to close this out -- given the amount of churn that's happened relevant to this issue it's probably best to start with a new issue if there's anything that still needs improvement.

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

No branches or pull requests

4 participants