Changeset 41756 for trunk/src/wp-content/themes/twentyseventeen/inc/template-tags.php

Timestamp:

10/04/2017 11:52:03 PM (8 years ago)

Author:

SergeyBiryukov

Message:

Bundled Themes: Change tag cloud format to a list (<ul>) for better semantics and accessibility.

List markup allows screen reader users to know in advance how many tags are within the list.

Props xkon, davidakennedy.
Fixes #40138.

File:

• trunk/src/wp-content/themes/twentyseventeen/inc/template-tags.php (modified) (1 diff)

trunk/src/wp-content/themes/twentyseventeen/inc/template-tags.php

@@ -85 +85 @@
                         }
 
-                        if ( $tags_list ) {
+                        if ( $tags_list && ! is_wp_error( $tags_list ) ) {
                             echo '<span class="tags-links">' . twentyseventeen_get_svg( array( 'icon' => 'hashtag' ) ) . '<span class="screen-reader-text">' . __( 'Tags', 'twentyseventeen' ) . '</span>' . $tags_list . '</span>';
                         }