Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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"]
.
Comments
Comment #1
jessebeach CreditAttribution: jessebeach commentedThis patch removes the redundant
[role=navigation]
attribute.Comment #2
mparker17Code looks good, works as advertised.
Looks good to me!
Comment #3
mgiffordThis makes sense to me.
Comment #4
mgiffordoops.
Comment #5
alexpottCommitted 51efbbf and pushed to 8.x. Thanks!
Comment #6
jessebeach CreditAttribution: jessebeach commentedAdding Spark tag for tracking.