[wp-trac] [WordPress Trac] #48170: Switch link list widgets to nav blocks
WordPress Trac
noreply at wordpress.org
Sun Jul 5 18:15:54 UTC 2020
#48170: Switch link list widgets to nav blocks
---------------------------------------------+----------------------------
Reporter: joedolson | Owner: audrasjb
Type: enhancement | Status: accepted
Priority: normal | Milestone: 5.5
Component: Widgets | Version:
Severity: normal | Resolution:
Keywords: has-patch needs-dev-note commit | Focuses: accessibility
---------------------------------------------+----------------------------
Comment (by joedolson):
Continuing to include `role="navigation"` is harmless, from a functional
standpoint, though it does throw warnings in HTML validation. It is
probably time for us to decide when we're going to remove the use of role
attributes to duplicate semantic functions like this, but I think that
should be coordinated with the accessibility-ready theme guidelines. I
think we should keep it for now, and make a plan for a timeline to remove
them on a global basis at some point in the future. (Excepting role=list
because Apple.)
Using 'navigation' as the property name kind of implies that this relates
to menus, I think. Can we use 'widgets'? Doing some research on that
seemed ambiguous; it's not listed as a property in the docs for
[https://developer.wordpress.org/reference/functions/add_theme_support/#html5
add_theme_support], but is listed as a property under
[https://codex.wordpress.org/Function_Reference/current_theme_supports
current_theme_supports] in the codex. In general, I'm inclined to believe
the new developer docs over info in the codex, but I couldn't find
anything about what 'widgets' html5 support might mean right now,
otherwise.
There are references in core to `current_theme_supports( 'widgets' )`, but
not to `current_theme_supports( 'html5', 'widgets' )`.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/48170#comment:23>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list