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

Explore Adopting EditContext API #207699

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Explore Adopting EditContext API #207699

wants to merge 1 commit into from

Conversation

hediet
Copy link
Member

@hediet hediet commented Mar 14, 2024

Missing functionality before this can be merged:

  • Setting to toggle between NativeEditContext and old TextArea based implementation (defaults to old TextArea implementation)
  • Screen-Reader support. I didn't figure out how to use the text area element for that, as editContext does not seem to work with it. So I guess we have to use the dom + dom selection. (basically the focused dom element has to have dom nodes that expose the text to the screen-reader, it also has to support text selection and update the selection so that screen-readers see what is selected. This dom node stays invisible though! Just for testing we can make it visible). We should definitely reach out to screen reader users to get feedback on this!
  • Copy Paste Support
  • Enter does not seem to work

@hediet hediet added this to the Backlog milestone Mar 14, 2024
@hediet hediet self-assigned this Mar 14, 2024
Signed-off-by: Henning Dieterichs <[email protected]>
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.

1 participant