Making WordPress.org

Opened 8 years ago

Closed 8 years ago

#2291 closed defect (bug) (fixed)

Misusage of aria-expanded in new plugin directory

Reported by: joedolson's profile joedolson Owned by: obenland's profile obenland
Milestone: Priority: normal
Component: Plugin Directory Keywords: needs-patch
Cc:

Description

The aria-expanded attribute is attached to the container div in the new directory; but the aria-expanded attribute needs to be on the controller, not on the controlled container. The attribute indicates whether the button you've focused on controls an element that's currently expanded or currently collapsed.

The aria-expanded attribute needs to be moved from the container div elements to the button in question.

Change History (2)

#1 @spacegrrl
8 years ago

To clarify: is this referring to a detail page for a plugin, not the plugin directory home page? Is this also in reference specifically to the Read More buttons on those detail pages, rather than the nav section (which seems to use aria-expanded correctly)?

Last edited 8 years ago by spacegrrl (previous) (diff)

#2 @obenland
8 years ago

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

In 4464:

Correct usage of aria-expanded attribute.

It belongs on the toggeling element, not the element that is being expanded.

Fixes #2291.

Note: See TracTickets for help on using tickets.