The header element of the page template of the Seven theme is marked up with a redundant [role="navigation"] ARIA landmark. The role of the header isn't navigation. It's a container. It can contain breadcrumbs, which are correctly marked up as [role="navigation"].

Screenshot of the Chrome dev tools showing the role=navigation attribute on the header element of the seven themes page template

Comments

jessebeach’s picture

Status:Active» Needs review
Issue tags:+accessibility, +TwinCities
StatusFileSize
new1.03 KB
PASSED: [[SimpleTest]]: [MySQL] 57,410 pass(es).
[ View ]

This patch removes the redundant [role=navigation] attribute.

mparker17’s picture

Status:Needs review» Reviewed & tested by the community

Code looks good, works as advertised.

Looks good to me!

mgifford’s picture

Status:Reviewed & tested by the community» Needs review

This makes sense to me.

mgifford’s picture

Status:Needs review» Reviewed & tested by the community

oops.

alexpott’s picture

Status:Reviewed & tested by the community» Fixed

Committed 51efbbf and pushed to 8.x. Thanks!

jessebeach’s picture

Issue tags:+Spark

Adding Spark tag for tracking.

Automatically closed -- issue fixed for 2 weeks with no activity.