Ticket #32028: 32028.3.patch

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

@@ -548 +548 @@
         background: rgba( 0, 0, 0, 0.05 );
         font-size: 16px;
         font-weight: normal;
+        display: inline-block;
+        line-height: 25px;
 }
 
 .tablenav .tablenav-pages a:hover,

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';
                 }
                 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;'
                 );