Making WordPress.org

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#3927 closed defect (bug) (fixed)

Remove the aria-expanded form the ul wporg-header-menu in the header menu

Reported by: rianrietveld's profile rianrietveld Owned by: obenland's profile obenland
Milestone: Priority: normal
Component: General Keywords:
Cc:

Description

The unordered list with the menu items in the header has an aria-expanded toggle. This can be removed.

The toggle should only be on the focusable element, in this case the button.
The open/closed state of the menu is announced when the button gets focus.
Adding it also to the ul is not necessary, because the ul doesn't get focus itself.

<ul id="wporg-header-menu" aria-expanded="false" class=" nav-menu">

Change History (3)

#1 @obenland
6 years ago

See [dotorg14645].

#2 @obenland
6 years ago

  • Owner set to obenland
  • Resolution set to fixed
  • Status changed from new to closed

In 7964:

WPorg: Remove unnecessary aria attribute.

Props rianrietveld.
Fixes #3927.

#3 @obenland
6 years ago

In 7965:

Themes: Remove unnecessary aria attribute.

Props rianrietveld.
See #3927.

Note: See TracTickets for help on using tickets.