Skip to content

Writing Flow: Unset typing flag if Escape pressed#10906

Merged
aduth merged 1 commit intomasterfrom
update/observe-typing-escape
Oct 26, 2018
Merged

Writing Flow: Unset typing flag if Escape pressed#10906
aduth merged 1 commit intomasterfrom
update/observe-typing-escape

Conversation

@aduth
Copy link
Copy Markdown
Member

@aduth aduth commented Oct 22, 2018

Extracted from #10699

This pull request seeks to resolve an issue where the contextual block toolbar is not accessible by keyboard when typing, except by artificially creating a selection for the mere purpose of deactivating the "is typing" application state flag. It does so by canceling the typing mode when the user presses Escape.

Implementation notes:

There are still some undesirable behaviors, of which this is not intended to cover, though I will create issues for:

Testing instructions:

Verify that, after typing in a paragraph block, pressing Escape causes the toolbar to be shown.

@aduth aduth added [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Feature] Writing Flow Block selection, navigation, splitting, merging, deletion... labels Oct 22, 2018
@youknowriad youknowriad added the Needs Accessibility Feedback Need input from accessibility label Oct 23, 2018
Copy link
Copy Markdown
Contributor

@youknowriad youknowriad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍 Code wise. Would be good to have a quick accessibility check.

@youknowriad
Copy link
Copy Markdown
Contributor

I'm still not sure what label we should use "Accessibility" or "Needs Accessibility Feedback" 🤷‍♂️

@aduth aduth mentioned this pull request Oct 24, 2018
4 tasks
@aduth aduth merged commit 0a3090d into master Oct 26, 2018
@youknowriad youknowriad deleted the update/observe-typing-escape branch October 26, 2018 13:01
@youknowriad youknowriad added this to the 4.2 milestone Oct 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Feature] Writing Flow Block selection, navigation, splitting, merging, deletion... [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). Needs Accessibility Feedback Need input from accessibility

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants