Ticket #32028: 32028.patch

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

@@ -495 +495 @@
         padding: 0 2px;
 }
 .tablenav-pages .current-page {
-        padding-top: 0;
+        padding-top: 2px;
         text-align: center;
+        font-size: 13px;
 }
 .tablenav-pages .next-page {
         margin-left: 2px;
@@ -525 +526 @@
         height: 30px;
         color: #555;
         line-height: 30px;
-        font-size: 12px;
 }
 
 .tablenav .no-pages,
@@ -565 +565 @@
 .tablenav .displaying-num {
         margin-right: 2px;
         color: #777;
-        font-size: 12px;
         font-style: italic;
 }
 
@@ -1551 +1550 @@
         .tablenav.bottom .displaying-num {
                 position: absolute;
                 right: 0;
-                top: 10px;
+                top: 5px;
                 font-size: 14px;
         }
 
@@ -1574 +1573 @@
                 height: 0;
         }
 
-        .tablenav-pages .pagination-links .paging-input {
-                font-size: 18px;
+        .tablenav-pages .pagination-links {
+                font-size: 16px;
         }
 
         .tablenav-pages .pagination-links a {
@@ -1585 +1584 @@
         }
 
         .tablenav-pages .pagination-links .current-page {
-                padding: 10px;
-                font-size: 14px;
+                padding: 9px 10px 10px;
+                font-size: 16px;
         }
 
         /* WP List Table Adjustments: General */

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

@@ -696 +696 @@
 
                 $page_links = array();
 
-                $disable_first = $disable_last = '';
+                $disable_first = $disable_last = $describedby_target_start = $describedby_target_end = '';
                 if ( $current == 1 ) {
                         $disable_first = ' disabled';
                 }
@@ -703 +703 @@
                 if ( $current == $total_pages ) {
                         $disable_last = ' disabled';
                 }
-                $page_links[] = sprintf( "<a class='%s' title='%s' href='%s'>%s</a>",
+                $page_links[] = sprintf( "<a class='%s' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
                         'first-page' . $disable_first,
-                        esc_attr__( 'Go to the first page' ),
                         esc_url( remove_query_arg( 'paged', $current_url ) ),
+                        __( 'First page' ),
                         '&laquo;'
                 );
 
-                $page_links[] = sprintf( "<a class='%s' title='%s' href='%s'>%s</a>",
+                $page_links[] = sprintf( "<a class='%s' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
                         'prev-page' . $disable_first,
-                        esc_attr__( 'Go to the previous page' ),
                         esc_url( add_query_arg( 'paged', max( 1, $current-1 ), $current_url ) ),
+                        __( 'Previous page' ),
                         '&lsaquo;'
                 );
 
                 if ( 'bottom' == $which ) {
                         $html_current_page = $current;
+                        $describedby_target_start = '<span id="table-paging">';
+                        $describedby_target_end = '</span>';
                 } else {
-                        $html_current_page = sprintf( "%s<input class='current-page' id='current-page-selector' title='%s' type='text' name='paged' value='%s' size='%d' />",
-                                '<label for="current-page-selector" class="screen-reader-text">' . __( 'Select Page' ) . '</label>',
-                                esc_attr__( 'Current page' ),
+                        $html_current_page = sprintf( "%s<input class='current-page' id='current-page-selector' type='text' name='paged' value='%s' size='%d' aria-describedby='table-paging' />",
+                                '<label for="current-page-selector" class="screen-reader-text">' . __( 'Current Page' ) . '</label>',
                                 $current,
                                 strlen( $total_pages )
                         );
                 }
                 $html_total_pages = sprintf( "<span class='total-pages'>%s</span>", number_format_i18n( $total_pages ) );
-                $page_links[] = '<span class="paging-input">' . sprintf( _x( '%1$s of %2$s', 'paging' ), $html_current_page, $html_total_pages ) . '</span>';
+                $page_links[] = $describedby_target_start . sprintf( _x( '%1$s of %2$s', 'paging' ), $html_current_page, $html_total_pages ) . $describedby_target_end;
 
-                $page_links[] = sprintf( "<a class='%s' title='%s' href='%s'>%s</a>",
+                $page_links[] = sprintf( "<a class='%s' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
                         'next-page' . $disable_last,
-                        esc_attr__( 'Go to the next page' ),
                         esc_url( add_query_arg( 'paged', min( $total_pages, $current+1 ), $current_url ) ),
+                        __( 'Next page' ),
                         '&rsaquo;'
                 );
 
-                $page_links[] = sprintf( "<a class='%s' title='%s' href='%s'>%s</a>",
+                $page_links[] = sprintf( "<a class='%s' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
                         'last-page' . $disable_last,
-                        esc_attr__( 'Go to the last page' ),
                         esc_url( add_query_arg( 'paged', $total_pages, $current_url ) ),
+                        __( 'Last page' ),
                         '&raquo;'
                 );