Skip to content

Speak: Add/complete audible feedback for link editing#10795

Merged
tofumatt merged 2 commits into
masterfrom
feature/speak-link-actions
Oct 23, 2018
Merged

Speak: Add/complete audible feedback for link editing#10795
tofumatt merged 2 commits into
masterfrom
feature/speak-link-actions

Conversation

@adamsilverstein

@adamsilverstein adamsilverstein commented Oct 19, 2018

Copy link
Copy Markdown
Member

Description

When users complete an action adding, editing or removing a link announce those action completions to assistive screen readers.

Fixes #1867

How has this been tested?

Adding and removing links in chrome with mac voice over enabled - the actions were properly announced.

Types of changes

  • Speak ‘link removed’ when a link is removed
  • Speak ‘link edited’ when a link is edited

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.

@adamsilverstein adamsilverstein force-pushed the feature/speak-link-actions branch from 94f692a to 2a9fb0c Compare October 19, 2018 13:55
@adamsilverstein adamsilverstein requested a review from mtias October 19, 2018 20:16
@mtias mtias added [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable labels Oct 20, 2018
@adamsilverstein adamsilverstein added this to the 4.2 milestone Oct 22, 2018

@tofumatt tofumatt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice, thanks!

@tofumatt tofumatt merged commit fa895eb into master Oct 23, 2018
@tofumatt tofumatt deleted the feature/speak-link-actions branch October 23, 2018 12:51
@ellatrix

Copy link
Copy Markdown
Member

No tests? :(

@tofumatt

Copy link
Copy Markdown
Member

My bad 😓, I've filed: #11006 and assigned it to me!

antpb pushed a commit to antpb/gutenberg that referenced this pull request Oct 26, 2018
* Speak ‘link removed’ when a link is removed

* Speak ‘link edited’ when a link is edited
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants