Make WordPress Core

Ticket #58703: 58703.diff

File 58703.diff, 12.7 KB (added by joedolson, 16 months ago)

Switch label/input order

  • src/wp-admin/css/list-tables.css

     
    570570        position: relative;
    571571}
    572572
    573 .check-column .label-covers-full-cell {
     573.check-column label {
    574574        width: 100%;
    575575        height: 100%;
    576576        display: block;
     
    579579        left: 0;
    580580}
    581581
    582 .check-column .label-covers-full-cell + input {
     582.check-column input {
    583583        position: relative;
    584584        z-index: 1;
    585585}
    586586
    587 .check-column .label-covers-full-cell:hover + input {
     587.check-column:hover input {
    588588        box-shadow: 0 0 0 1px #2271b1;
    589589}
    590590
  • src/wp-admin/includes/class-wp-comments-list-table.php

     
    895895
    896896                if ( $this->user_can ) {
    897897                        ?>
    898                 <label class="label-covers-full-cell" for="cb-select-<?php echo $comment->comment_ID; ?>">
     898                <input id="cb-select-<?php echo $comment->comment_ID; ?>" type="checkbox" name="delete_comments[]" value="<?php echo $comment->comment_ID; ?>" />
     899                <label for="cb-select-<?php echo $comment->comment_ID; ?>">
    899900                        <span class="screen-reader-text">
    900901                        <?php
    901902                        /* translators: Hidden accessibility text. */
     
    903904                        ?>
    904905                        </span>
    905906                </label>
    906                 <input id="cb-select-<?php echo $comment->comment_ID; ?>" type="checkbox" name="delete_comments[]" value="<?php echo $comment->comment_ID; ?>" />
    907907                        <?php
    908908                }
    909909        }
  • src/wp-admin/includes/class-wp-links-list-table.php

     
    174174                $link = $item;
    175175
    176176                ?>
    177                 <label class="label-covers-full-cell" for="cb-select-<?php echo $link->link_id; ?>">
     177                <input type="checkbox" name="linkcheck[]" id="cb-select-<?php echo $link->link_id; ?>" value="<?php echo esc_attr( $link->link_id ); ?>" />
     178                <label for="cb-select-<?php echo $link->link_id; ?>">
    178179                        <span class="screen-reader-text">
    179180                        <?php
    180181                        /* translators: Hidden accessibility text. %s: Link name. */
     
    182183                        ?>
    183184                        </span>
    184185                </label>
    185                 <input type="checkbox" name="linkcheck[]" id="cb-select-<?php echo $link->link_id; ?>" value="<?php echo esc_attr( $link->link_id ); ?>" />
    186186                <?php
    187187        }
    188188
  • src/wp-admin/includes/class-wp-list-table.php

     
    13341334
    13351335                if ( ! empty( $columns['cb'] ) ) {
    13361336                        static $cb_counter = 1;
    1337                         $columns['cb']     = '<label class="label-covers-full-cell" for="cb-select-all-' . $cb_counter . '">' .
     1337                        $columns['cb']     = '<input id="cb-select-all-' . $cb_counter . '" type="checkbox" />
     1338                        <label for="cb-select-all-' . $cb_counter . '">' .
    13381339                                '<span class="screen-reader-text">' .
    13391340                                        /* translators: Hidden accessibility text. */
    13401341                                        __( 'Select All' ) .
    13411342                                '</span>' .
    1342                                 '</label>' .
    1343                                 '<input id="cb-select-all-' . $cb_counter . '" type="checkbox" />';
     1343                                '</label>';
    13441344                        $cb_counter++;
    13451345                }
    13461346
  • src/wp-admin/includes/class-wp-media-list-table.php

     
    419419
    420420                if ( current_user_can( 'edit_post', $post->ID ) ) {
    421421                        ?>
    422                         <label class="label-covers-full-cell" for="cb-select-<?php echo $post->ID; ?>">
     422                        <input type="checkbox" name="media[]" id="cb-select-<?php echo $post->ID; ?>" value="<?php echo $post->ID; ?>" />
     423                        <label for="cb-select-<?php echo $post->ID; ?>">
    423424                                <span class="screen-reader-text">
    424425                                <?php
    425426                                /* translators: Hidden accessibility text. %s: Attachment title. */
     
    427428                                ?>
    428429                                </span>
    429430                        </label>
    430                         <input type="checkbox" name="media[]" id="cb-select-<?php echo $post->ID; ?>" value="<?php echo $post->ID; ?>" />
    431431                        <?php
    432432                }
    433433        }
  • src/wp-admin/includes/class-wp-ms-sites-list-table.php

     
    420420                if ( ! is_main_site( $blog['blog_id'] ) ) :
    421421                        $blogname = untrailingslashit( $blog['domain'] . $blog['path'] );
    422422                        ?>
    423                         <label class="label-covers-full-cell" for="blog_<?php echo $blog['blog_id']; ?>">
     423                        <input type="checkbox" id="blog_<?php echo $blog['blog_id']; ?>" name="allblogs[]"
     424                                value="<?php echo esc_attr( $blog['blog_id'] ); ?>" />
     425                        <label for="blog_<?php echo $blog['blog_id']; ?>">
    424426                                <span class="screen-reader-text">
    425427                                <?php
    426428                                /* translators: %s: Site URL. */
     
    428430                                ?>
    429431                                </span>
    430432                        </label>
    431                         <input type="checkbox" id="blog_<?php echo $blog['blog_id']; ?>" name="allblogs[]"
    432                                 value="<?php echo esc_attr( $blog['blog_id'] ); ?>" />
    433433                        <?php
    434434                endif;
    435435        }
  • src/wp-admin/includes/class-wp-ms-themes-list-table.php

     
    512512                $theme       = $item;
    513513                $checkbox_id = 'checkbox_' . md5( $theme->get( 'Name' ) );
    514514                ?>
    515                 <label class="label-covers-full-cell" for="<?php echo $checkbox_id; ?>" >
     515                <input type="checkbox" name="checked[]" value="<?php echo esc_attr( $theme->get_stylesheet() ); ?>" id="<?php echo $checkbox_id; ?>" />
     516                <label for="<?php echo $checkbox_id; ?>" >
    516517                        <span class="screen-reader-text">
    517518                        <?php
    518519                        printf(
     
    523524                        ?>
    524525                        </span>
    525526                </label>
    526                 <input type="checkbox" name="checked[]" value="<?php echo esc_attr( $theme->get_stylesheet() ); ?>" id="<?php echo $checkbox_id; ?>" />
    527527                <?php
    528528        }
    529529
  • src/wp-admin/includes/class-wp-ms-users-list-table.php

     
    235235                        return;
    236236                }
    237237                ?>
    238                 <label class="label-covers-full-cell" for="blog_<?php echo $user->ID; ?>">
     238                <input type="checkbox" id="blog_<?php echo $user->ID; ?>" name="allusers[]" value="<?php echo esc_attr( $user->ID ); ?>" />
     239                <label for="blog_<?php echo $user->ID; ?>">
    239240                        <span class="screen-reader-text">
    240241                        <?php
    241242                        /* translators: Hidden accessibility text. %s: User login. */
     
    243244                        ?>
    244245                        </span>
    245246                </label>
    246                 <input type="checkbox" id="blog_<?php echo $user->ID; ?>" name="allusers[]" value="<?php echo esc_attr( $user->ID ); ?>" />
    247247                <?php
    248248        }
    249249
  • src/wp-admin/includes/class-wp-plugins-list-table.php

     
    993993                        $checkbox = '';
    994994                } else {
    995995                        $checkbox = sprintf(
    996                                 '<label class="label-covers-full-cell" for="%1$s"><span class="screen-reader-text">%2$s</span></label>' .
    997                                 '<input type="checkbox" name="checked[]" value="%3$s" id="%1$s" />',
     996                                '<input type="checkbox" name="checked[]" value="%3$s" id="%1$s" />' .
     997                                '<label for="%1$s"><span class="screen-reader-text">%2$s</span></label>',
    998998                                $checkbox_id,
    999999                                /* translators: Hidden accessibility text. %s: Plugin name. */
    10001000                                sprintf( __( 'Select %s' ), $plugin_data['Name'] ),
  • src/wp-admin/includes/class-wp-posts-list-table.php

     
    10281028                 */
    10291029                if ( apply_filters( 'wp_list_table_show_post_checkbox', $show, $post ) ) :
    10301030                        ?>
    1031                         <label class="label-covers-full-cell" for="cb-select-<?php the_ID(); ?>">
     1031                        <input id="cb-select-<?php the_ID(); ?>" type="checkbox" name="post[]" value="<?php the_ID(); ?>" />
     1032                        <label for="cb-select-<?php the_ID(); ?>">
    10321033                                <span class="screen-reader-text">
    10331034                                <?php
    10341035                                        /* translators: %s: Post title. */
     
    10361037                                ?>
    10371038                                </span>
    10381039                        </label>
    1039                         <input id="cb-select-<?php the_ID(); ?>" type="checkbox" name="post[]" value="<?php the_ID(); ?>" />
    10401040                        <div class="locked-indicator">
    10411041                                <span class="locked-indicator-icon" aria-hidden="true"></span>
    10421042                                <span class="screen-reader-text">
  • src/wp-admin/includes/class-wp-privacy-requests-table.php

     
    416416         */
    417417        public function column_cb( $item ) {
    418418                return sprintf(
    419                         '<label class="label-covers-full-cell" for="requester_%1$s"><span class="screen-reader-text">%2$s</span></label>' .
    420                         '<input type="checkbox" name="request_id[]" id="requester_%1$s" value="%1$s" /><span class="spinner"></span>',
     419                        '<input type="checkbox" name="request_id[]" id="requester_%1$s" value="%1$s" /><span class="spinner"></span>' .
     420                        '<label for="requester_%1$s"><span class="screen-reader-text">%2$s</span></label>',
    421421                        esc_attr( $item->ID ),
    422422                        /* translators: Hidden accessibility text. %s: Email address. */
    423423                        sprintf( __( 'Select %s' ), $item->email )
  • src/wp-admin/includes/class-wp-terms-list-table.php

     
    365365
    366366                if ( current_user_can( 'delete_term', $tag->term_id ) ) {
    367367                        return sprintf(
    368                                 '<label class="label-covers-full-cell" for="cb-select-%1$s"><span class="screen-reader-text">%2$s</span></label>' .
    369                                 '<input type="checkbox" name="delete_tags[]" value="%1$s" id="cb-select-%1$s" />',
     368                                '<input type="checkbox" name="delete_tags[]" value="%1$s" id="cb-select-%1$s" />' .
     369                                '<label for="cb-select-%1$s"><span class="screen-reader-text">%2$s</span></label>',
    370370                                $tag->term_id,
    371371                                /* translators: Hidden accessibility text. %s: Taxonomy term name. */
    372372                                sprintf( __( 'Select %s' ), $tag->name )
  • src/wp-admin/includes/class-wp-users-list-table.php

     
    524524
    525525                        // Set up the checkbox (because the user is editable, otherwise it's empty).
    526526                        $checkbox = sprintf(
    527                                 '<label class="label-covers-full-cell" for="user_%1$s"><span class="screen-reader-text">%2$s</span></label>' .
    528                                 '<input type="checkbox" name="users[]" id="user_%1$s" class="%3$s" value="%1$s" />',
     527                                '<input type="checkbox" name="users[]" id="user_%1$s" class="%3$s" value="%1$s" />' .
     528                                '<label for="user_%1$s"><span class="screen-reader-text">%2$s</span></label>',
    529529                                $user_object->ID,
    530530                                /* translators: Hidden accessibility text. %s: User login. */
    531531                                sprintf( __( 'Select %s' ), $user_object->user_login ),
  • src/wp-admin/update-core.php

     
    571571        <tr>
    572572                <td class="check-column">
    573573                        <?php if ( $compatible_php ) : ?>
    574                                 <label for="<?php echo $checkbox_id; ?>" class="label-covers-full-cell">
     574                                <label for="<?php echo $checkbox_id; ?>">
    575575                                        <span class="screen-reader-text">
    576576                                        <?php
    577577                                        /* translators: Hidden accessibility text. %s: Plugin name. */
     
    749749        <tr>
    750750                <td class="check-column">
    751751                        <?php if ( $compatible_wp && $compatible_php ) : ?>
    752                                 <label for="<?php echo $checkbox_id; ?>" class="label-covers-full-cell">
     752                                <label for="<?php echo $checkbox_id; ?>">
    753753                                        <span class="screen-reader-text">
    754754                                        <?php
    755755                                        /* translators: Hidden accessibility text. %s: Theme name. */