Making WordPress.org

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#4131 closed enhancement (fixed)

Add aria-current to global menu

Reported by: otto42's profile Otto42 Owned by:
Milestone: Priority: normal
Component: General Keywords:
Cc:

Description

As mentioned in https://meta.trac.wordpress.org/ticket/4091#comment:12 by @adrian-roselli

While I am here, is there an issue somewhere to add aria-current to the currently active link in the navigation? The screen reader should announce that I am on the Plugins page.

This seems useful. As it is, the class="current" is added to the link for styling.

Would adding aria-current to the same link be the correct approach here? Or should it be added elsewhere in the list?

Change History (2)

#1 @Otto42
6 years ago

From reading https://tink.uk/using-the-aria-current-attribute/ it seems like aria-current="page" or aria-current="location" could be appropriate.

The safe bet seems to be aria-current="true", so we'll go with that for now. If something else makes more sense, let us know.

#2 @Otto42
6 years ago

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

Fixed in [dotorg:14782].

Last edited 6 years ago by Otto42 (previous) (diff)
Note: See TracTickets for help on using tickets.