[wp-trac] [WordPress Trac] #54079: Remove "role" attribute on HTML5 elements with a default landmark role.
WordPress Trac
noreply at wordpress.org
Fri Oct 29 06:42:40 UTC 2021
#54079: Remove "role" attribute on HTML5 elements with a default landmark role.
-------------------------+-------------------------------------------------
Reporter: costdev | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: 5.9
Component: Bundled | Version:
Theme |
Severity: normal | Resolution:
Keywords: has-patch | Focuses: accessibility, css, coding-
| standards
-------------------------+-------------------------------------------------
Changes (by costdev):
* keywords: has-patch needs-refresh 2nd-opinion => has-patch
Comment:
Patch refreshed with two options:
1. [https://core.trac.wordpress.org/attachment/ticket/54079/54079.1.diff
54079.1] - Keep `role="contentinfo"` for Twenty Twelve `footer`.
2. [https://core.trac.wordpress.org/attachment/ticket/54079/54079.2.diff
54079.2] - The same as above, but also add `class="contentinfo"` along
with adjustments to Twenty Twelve's `style.css`. The intention here would
be to phase out the targeting of `footer[role="contentinfo"]` by child
themes, who will hopefully opt for the `footer.contentinfo` class selector
instead.
I don't have a preference between the patches. We have to keep the
`role="contentinfo"` on the Twenty Twelve `footer`, so I'm happy for
either a Component Maintainer or Committer to make the final call on which
of the two latest patches is the better choice.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/54079#comment:14>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list