WordPress.org
Get WordPress

Make WordPress Core

Changeset 56261


Ignore:
Timestamp:
07/18/2023 02:07:02 PM (20 months ago)
Author:
SergeyBiryukov
Message:

Administration: Add a missing closing </span> tag for column sorting indicators.

Includes wrapping a few other long markup lines for better readability.

Follow-up to [55971], [56004], [56260].

See #32170, #57839.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-admin/includes/class-wp-list-table.php

    r56260 r56261  
    774774
    775775            printf(
    776                 "<a href='%s' class='%s' id='view-switch-$mode'$aria_current><span class='screen-reader-text'>%s</span></a>\n",
     776                "<a href='%s' class='%s' id='view-switch-$mode'$aria_current>" .
     777                    "<span class='screen-reader-text'>%s</span>" .
     778                "</a>\n",
    777779                esc_url( remove_query_arg( 'attachment-filter', add_query_arg( 'mode', $mode ) ) ),
    778780                implode( ' ', $classes ),
    … …  
    820822            // No comments at all.
    821823            printf(
    822                 '<span aria-hidden="true">&#8212;</span><span class="screen-reader-text">%s</span>',
     824                '<span aria-hidden="true">&#8212;</span>' .
     825                '<span class="screen-reader-text">%s</span>',
    823826                __( 'No comments' )
    824827            );
    … …  
    826829            // Don't link the comment bubble for a trashed post.
    827830            printf(
    828                 '<span class="post-com-count post-com-count-approved"><span class="comment-count-approved" aria-hidden="true">%s</span><span class="screen-reader-text">%s</span></span>',
     831                '<span class="post-com-count post-com-count-approved">' .
     832                    '<span class="comment-count-approved" aria-hidden="true">%s</span>' .
     833                    '<span class="screen-reader-text">%s</span>' .
     834                '</span>',
    829835                $approved_comments_number,
    830836                $pending_comments ? $approved_phrase : $approved_only_phrase
    … …  
    833839            // Link the comment bubble to approved comments.
    834840            printf(
    835                 '<a href="%s" class="post-com-count post-com-count-approved"><span class="comment-count-approved" aria-hidden="true">%s</span><span class="screen-reader-text">%s</span></a>',
     841                '<a href="%s" class="post-com-count post-com-count-approved">' .
     842                    '<span class="comment-count-approved" aria-hidden="true">%s</span>' .
     843                    '<span class="screen-reader-text">%s</span>' .
     844                '</a>',
    836845                esc_url(
    837846                    add_query_arg(
    … …  
    849858            // Don't link the comment bubble when there are no approved comments.
    850859            printf(
    851                 '<span class="post-com-count post-com-count-no-comments"><span class="comment-count comment-count-no-comments" aria-hidden="true">%s</span><span class="screen-reader-text">%s</span></span>',
     860                '<span class="post-com-count post-com-count-no-comments">' .
     861                    '<span class="comment-count comment-count-no-comments" aria-hidden="true">%s</span>' .
     862                    '<span class="screen-reader-text">%s</span>' .
     863                '</span>',
    852864                $approved_comments_number,
    853865                $pending_comments ?
    … …  
    861873        if ( $pending_comments ) {
    862874            printf(
    863                 '<a href="%s" class="post-com-count post-com-count-pending"><span class="comment-count-pending" aria-hidden="true">%s</span><span class="screen-reader-text">%s</span></a>',
     875                '<a href="%s" class="post-com-count post-com-count-pending">' .
     876                    '<span class="comment-count-pending" aria-hidden="true">%s</span>' .
     877                    '<span class="screen-reader-text">%s</span>' .
     878                '</a>',
    864879                esc_url(
    865880                    add_query_arg(
    … …  
    876891        } else {
    877892            printf(
    878                 '<span class="post-com-count post-com-count-pending post-com-count-no-pending"><span class="comment-count comment-count-no-pending" aria-hidden="true">%s</span><span class="screen-reader-text">%s</span></span>',
     893                '<span class="post-com-count post-com-count-pending post-com-count-no-pending">' .
     894                    '<span class="comment-count comment-count-no-pending" aria-hidden="true">%s</span>' .
     895                    '<span class="screen-reader-text">%s</span>' .
     896                '</span>',
    879897                $pending_comments_number,
    880898                $approved_comments ?
    … …  
    10051023        } else {
    10061024            $page_links[] = sprintf(
    1007                 "<a class='first-page button' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
     1025                "<a class='first-page button' href='%s'>" .
     1026                    "<span class='screen-reader-text'>%s</span>" .
     1027                    "<span aria-hidden='true'>%s</span>" .
     1028                '</a>',
    10081029                esc_url( remove_query_arg( 'paged', $current_url ) ),
    10091030                /* translators: Hidden accessibility text. */
    … …  
    10171038        } else {
    10181039            $page_links[] = sprintf(
    1019                 "<a class='prev-page button' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
     1040                "<a class='prev-page button' href='%s'>" .
     1041                    "<span class='screen-reader-text'>%s</span>" .
     1042                    "<span aria-hidden='true'>%s</span>" .
     1043                '</a>',
    10201044                esc_url( add_query_arg( 'paged', max( 1, $current - 1 ), $current_url ) ),
    10211045                /* translators: Hidden accessibility text. */
    … …  
    10271051        if ( 'bottom' === $which ) {
    10281052            $html_current_page  = $current;
    1029             $total_pages_before = '<span class="screen-reader-text">' .
     1053            $total_pages_before = sprintf(
     1054                '<span class="screen-reader-text">%s</span>' .
     1055                '<span id="table-paging" class="paging-input">' .
     1056                '<span class="tablenav-paging-text">',
    10301057                /* translators: Hidden accessibility text. */
    1031                 __( 'Current Page' ) .
    1032             '</span><span id="table-paging" class="paging-input"><span class="tablenav-paging-text">';
     1058                __( 'Current Page' )
     1059            );
    10331060        } else {
    10341061            $html_current_page = sprintf(
    1035                 "%s<input class='current-page' id='current-page-selector' type='text' name='paged' value='%s' size='%d' aria-describedby='table-paging' /><span class='tablenav-paging-text'>",
    1036                 '<label for="current-page-selector" class="screen-reader-text">' .
    1037                     /* translators: Hidden accessibility text. */
    1038                     __( 'Current Page' ) .
    1039                 '</label>',
     1062                '<label for="current-page-selector" class="screen-reader-text">%s</label>' .
     1063                "<input class='current-page' id='current-page-selector' type='text'
     1064                    name='paged' value='%s' size='%d' aria-describedby='table-paging' />" .
     1065                "<span class='tablenav-paging-text'>",
     1066                /* translators: Hidden accessibility text. */
     1067                __( 'Current Page' ),
    10401068                $current,
    10411069                strlen( $total_pages )
    10421070            );
    10431071        }
     1072
    10441073        $html_total_pages = sprintf( "<span class='total-pages'>%s</span>", number_format_i18n( $total_pages ) );
    1045         $page_links[]     = $total_pages_before . sprintf(
     1074
     1075        $page_links[] = $total_pages_before . sprintf(
    10461076            /* translators: 1: Current page, 2: Total pages. */
    10471077            _x( '%1$s of %2$s', 'paging' ),
    … …  
    10541084        } else {
    10551085            $page_links[] = sprintf(
    1056                 "<a class='next-page button' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
     1086                "<a class='next-page button' href='%s'>" .
     1087                    "<span class='screen-reader-text'>%s</span>" .
     1088                    "<span aria-hidden='true'>%s</span>" .
     1089                '</a>',
    10571090                esc_url( add_query_arg( 'paged', min( $total_pages, $current + 1 ), $current_url ) ),
    10581091                /* translators: Hidden accessibility text. */
    … …  
    10661099        } else {
    10671100            $page_links[] = sprintf(
    1068                 "<a class='last-page button' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
     1101                "<a class='last-page button' href='%s'>" .
     1102                    "<span class='screen-reader-text'>%s</span>" .
     1103                    "<span aria-hidden='true'>%s</span>" .
     1104                '</a>',
    10691105                esc_url( add_query_arg( 'paged', $total_pages, $current_url ) ),
    10701106                /* translators: Hidden accessibility text. */
    … …  
    14221458
    14231459                // Print an 'abbr' attribute if a value is provided via get_sortable_columns().
    1424                 $abbr_attr           = $abbr ? ' abbr="' . esc_attr( $abbr ) . '"' : '';
    1425                 $column_display_name = '<a href="' . esc_url( add_query_arg( compact( 'orderby', 'order' ), $current_url ) ) . '"><span>' . $column_display_name . '</span><span class="sorting-indicators"><span class="sorting-indicator asc" aria-hidden="true"></span><span class="sorting-indicator desc" aria-hidden="true"></span>' . $order_text . '</a>';
     1460                $abbr_attr = $abbr ? ' abbr="' . esc_attr( $abbr ) . '"' : '';
     1461
     1462                $column_display_name = sprintf(
     1463                    '<a href="%1$s">' .
     1464                        '<span>%2$s</span>' .
     1465                        '<span class="sorting-indicators">' .
     1466                            '<span class="sorting-indicator asc" aria-hidden="true"></span>' .
     1467                            '<span class="sorting-indicator desc" aria-hidden="true"></span>' .
     1468                        '</span>' .
     1469                        '%3$s' .
     1470                    '</a>',
     1471                    esc_url( add_query_arg( compact( 'orderby', 'order' ), $current_url ) ),
     1472                    $column_display_name,
     1473                    $order_text
     1474                );
    14261475            }
    14271476
Note: See TracChangeset for help on using the changeset viewer.