WordPress.org
Get WordPress

Make WordPress Core

Ticket #40187: 40187.diff

File 40187.diff, 3.9 KB (added by audrasjb, 7 years ago)

Tag cloud list output, #40187

  • src/wp-admin/css/edit.css 

    diff --git a/src/wp-admin/css/edit.css b/src/wp-admin/css/edit.css
index 64b98ae..1595983 100644
    a b span.wp-media-buttons-icon:before { 
    574574        position: absolute;
    575575}
    576576
    577 .tagchecklist > span {
     577.tagchecklist > li {
     578        display: inline-block;
    578579        float: left;
    579580        margin-right: 25px;
    580581        font-size: 13px;
    … … table.links-table { 
    14401441                margin: 25px 10px;
    14411442        }
    14421443
    1443         .tagchecklist > span {
     1444        .tagchecklist > li {
    14441445                font-size: 16px;
    14451446                line-height: 1.4;
    14461447        }

  • src/wp-admin/css/press-this.css 

    diff --git a/src/wp-admin/css/press-this.css b/src/wp-admin/css/press-this.css
index bafea24..5227f9d 100644
    a b dd { 
    765765
    766766/* Tags */
    767767.tagchecklist {
     768        margin: 0;
    768769        padding: 16px 28px 5px;
     770        list-style: none;
    769771}
    770772
    771773.tagchecklist:before,
    … … dd { 
    778780        clear: both;
    779781}
    780782
    781 .tagchecklist > span {
     783.tagchecklist > li {
     784        display: inline-block;
    782785        float: left;
    783786        margin-right: 25px;
    784787        font-size: 13px;
    … … dd { 
    788791}
    789792
    790793@media (max-width: 600px) {
    791         .tagchecklist > span {
     794        .tagchecklist > li {
    792795                margin-bottom: 15px;
    793796                font-size: 16px;
    794797                line-height: 1.3;

  • src/wp-admin/includes/class-wp-press-this.php 

    diff --git a/src/wp-admin/includes/class-wp-press-this.php b/src/wp-admin/includes/class-wp-press-this.php
index 1a30e41..86d5106 100644
    a b class WP_Press_This { 
    950950
    951951                        ?>
    952952                        </div>
    953                         <div class="tagchecklist"></div>
     953                        <ul class="tagchecklist"></ul>
    954954                </div>
    955955                <?php
    956956

  • src/wp-admin/includes/meta-boxes.php 

    diff --git a/src/wp-admin/includes/meta-boxes.php b/src/wp-admin/includes/meta-boxes.php
index 1d9b283..104cb64 100644
    a b function post_tags_meta_box( $post, $box ) { 
    453453                <p><?php echo $taxonomy->labels->no_terms; ?></p>
    454454        <?php endif; ?>
    455455        </div>
    456         <div class="tagchecklist"></div>
     456        <ul class="tagchecklist"></ul>
    457457</div>
    458458<?php if ( $user_can_assign_terms ) : ?>
    459459<p class="hide-if-no-js"><button type="button" class="button-link tagcloud-link" id="link-<?php echo $tax_name; ?>" aria-expanded="false"><?php echo $taxonomy->labels->choose_from_most_used; ?></button></p>

  • src/wp-admin/js/tags-box.js 

    diff --git a/src/wp-admin/js/tags-box.js b/src/wp-admin/js/tags-box.js
index 3867c7d..bcfa72c 100644
    a b var tagBox, array_unique_noempty; 
    7474                        tagchecklist.empty();
    7575
    7676                        $.each( current_tags, function( key, val ) {
    77                                 var span, xbutton;
     77                                var item, xbutton;
    7878
    7979                                val = $.trim( val );
    8080
    8181                                if ( ! val )
    8282                                        return;
    8383
    84                                 // Create a new span, and ensure the text is properly escaped.
    85                                 span = $('<span />').text( val );
     84                                // Create a new item, and ensure the text is properly escaped.
     85                                item = $('<li />').text( val );
    8686
    8787                                // If tags editing isn't disabled, create the X button.
    8888                                if ( ! disabled ) {
    … … var tagBox, array_unique_noempty; 
    9292                                         */
    9393                                        xbutton = $( '<button type="button" id="' + id + '-check-num-' + key + '" class="ntdelbutton">' +
    9494                                                '<span class="remove-tag-icon" aria-hidden="true"></span>' +
    95                                                 '<span class="screen-reader-text">' + window.tagsSuggestL10n.removeTerm + ' ' + span.html() + '</span>' +
     95                                                '<span class="screen-reader-text">' + window.tagsSuggestL10n.removeTerm + ' ' + item.html() + '</span>' +
    9696                                                '</button>' );
    9797
    9898                                        xbutton.on( 'click keypress', function( e ) {
    … … var tagBox, array_unique_noempty; 
    112112                                                }
    113113                                        });
    114114
    115                                         span.prepend( '&nbsp;' ).prepend( xbutton );
     115                                        item.prepend( '&nbsp;' ).prepend( xbutton );
    116116                                }
    117117
    118                                 // Append the span to the tag list.
    119                                 tagchecklist.append( span );
     118                                // Append the item to the tag list.
     119                                tagchecklist.append( item );
    120120                        });
    121121                        // The buttons list is built now, give feedback to screen reader users.
    122122                        tagBox.screenReadersMessage();