Skip to content

Improve successful draft save notification for screen readers#37683

Merged
kevin940726 merged 3 commits into
WordPress:trunkfrom
alexstine:fix/save-draft-notice-accessibility
Jan 3, 2022
Merged

Improve successful draft save notification for screen readers#37683
kevin940726 merged 3 commits into
WordPress:trunkfrom
alexstine:fix/save-draft-notice-accessibility

Conversation

@alexstine

@alexstine alexstine commented Jan 3, 2022 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

I modified the success save draft notification to be more descriptive for screen readers.

How has this been tested?

Open any Post or Page and select Save draft. Observe how screen readers now announce Draft saved instead of just Saved.

Screenshots

Types of changes

Bug fix

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • I've tested my changes with keyboard and screen readers.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.
  • I've updated all React Native files affected by any refactorings/renamings in this PR (please manually search all *.native.js files for terms that need renaming or removal).

@alexstine alexstine self-assigned this Jan 3, 2022
@alexstine alexstine added [Package] Editor /packages/editor [Type] Enhancement A suggestion for improvement. [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). labels Jan 3, 2022
@alexstine

alexstine commented Jan 3, 2022

Copy link
Copy Markdown
Contributor Author

As usual, I have no idea why the tests are failing now. Looks unrelated to this PR.

kevin940726
kevin940726 approved these changes Jan 3, 2022
View reviewed changes

@kevin940726 kevin940726 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.

Changes make sense to me 👍 .

I restarted the tests and they passed.

@kevin940726 kevin940726 merged commit e68e4ca into WordPress:trunk Jan 3, 2022
@github-actions github-actions Bot added this to the Gutenberg 12.4 milestone Jan 3, 2022
@alexstine

alexstine commented Jan 3, 2022

Copy link
Copy Markdown
Contributor Author

Thanks @kevin940726 for the quick review! 💯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kevin940726 kevin940726 kevin940726 approved these changes
@youknowriad youknowriad Awaiting requested review from youknowriad
@talldan talldan Awaiting requested review from talldan
@aristath aristath Awaiting requested review from aristath

Assignees

@alexstine alexstine

Labels

[Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). [Package] Editor /packages/editor [Type] Enhancement A suggestion for improvement.

Projects

None yet

Milestone

Gutenberg 12.4

Development

Successfully merging this pull request may close these issues.

2 participants

@alexstine @kevin940726