Skip to content

Fix/issue 12501 menu item aria label#12955

Merged
jorgefilipecosta merged 1 commit intoWordPress:masterfrom
timwright12:fix/issue-12501-MenuItem-aria-label
Feb 1, 2019
Merged

Fix/issue 12501 menu item aria label#12955
jorgefilipecosta merged 1 commit intoWordPress:masterfrom
timwright12:fix/issue-12501-MenuItem-aria-label

Conversation

@timwright12
Copy link
Copy Markdown
Contributor

@timwright12 timwright12 commented Dec 17, 2018

Description

Fixes #12501

How has this been tested?

  • Local unit tests
  • Local e2e tests
  • Browser testing

Screenshots

screen shot 2018-12-17 at 1 34 59 pm

Types of changes

Removed aria-label on the menuItem component because it was redundant with the button text itself. Also update the tests and snapshots to match the new output.

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.

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

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants