#51455 closed defect (bug) (fixed)
Add theme support to make tag cloud widget output list of links wrapped within a <nav> element
Reported by: | audrasjb | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.6 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Widgets | Keywords: | has-patch has-dev-note |
Focuses: | accessibility | Cc: |
Description
Follow-up to https://core.trac.wordpress.org/ticket/48170#comment:36
Tag cloud widget was missed from #48170.
Attachments (2)
Change History (9)
#1
@
4 years ago
- Keywords has-patch added; needs-patch removed
51455.diff
addresses this issue.
Please don't forget to give proper props to @justinahinon who raised the issue in https://core.trac.wordpress.org/ticket/48170#comment:36
#2
@
4 years ago
@audrasjb @justinahinon
Why not use this code
$title = trim( strip_tags( $title ) ); $aria_label = $title ? $title : $default_title;
Change With
$title = wp_strip_all_tags( $title ); $aria_label = $title ? $title : $default_title;
#3
@
4 years ago
@ravipatel I'd prefer to keep consistency with the other existing widgets code (see category widget for reference) :)
This ticket was mentioned in Slack in #accessibility by audrasjb. View the logs.
4 years ago
#5
@
4 years ago
- Owner set to SergeyBiryukov
- Resolution set to fixed
- Status changed from new to closed
In 49177:
This ticket was mentioned in Slack in #core by daisyo. View the logs.
4 years ago
Note: See
TracTickets for help on using
tickets.
Accessibility: Widgets: Add theme support to make tag cloud widget output list of links wrapped within a
<nav>
element.