Make WordPress Core

Changeset 56261


Ignore:
Timestamp:
07/18/2023 02:07:02 PM (3 years 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.