Changeset 43019 for trunk/src/wp-admin/includes/class-wp-list-table.php

Timestamp:

04/29/2018 04:24:41 PM (7 years ago)

Author:

afercia

Message:

Accessibility: Better color contrast for the tables pagination links.

By making the pagination links use the default style for buttons, color contrast
is improved and the CSS is simplified. Improves consistency with other UI controls.
Also, slightly increases the white space below the pagination links.

Props benoitchantre, afercia.
Fixes #41858.

File:

• trunk/src/wp-admin/includes/class-wp-list-table.php (modified) (4 diffs)

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

@@ -818 +818 @@
 
         if ( $disable_first ) {
-            $page_links[] = '<span class="tablenav-pages-navspan" aria-hidden="true">&laquo;</span>';
+            $page_links[] = '<span class="tablenav-pages-navspan button disabled" aria-hidden="true">&laquo;</span>';
         } else {
             $page_links[] = sprintf(
-                "<a class='first-page' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
+                "<a class='first-page button' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
                 esc_url( remove_query_arg( 'paged', $current_url ) ),
                 __( 'First page' ),
@@ -829 +829 @@
 
         if ( $disable_prev ) {
-            $page_links[] = '<span class="tablenav-pages-navspan" aria-hidden="true">&lsaquo;</span>';
+            $page_links[] = '<span class="tablenav-pages-navspan button disabled" aria-hidden="true">&lsaquo;</span>';
         } else {
             $page_links[] = sprintf(
-                "<a class='prev-page' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
+                "<a class='prev-page button' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
                 esc_url( add_query_arg( 'paged', max( 1, $current - 1 ), $current_url ) ),
                 __( 'Previous page' ),
@@ -854 +854 @@
 
         if ( $disable_next ) {
-            $page_links[] = '<span class="tablenav-pages-navspan" aria-hidden="true">&rsaquo;</span>';
+            $page_links[] = '<span class="tablenav-pages-navspan button disabled" aria-hidden="true">&rsaquo;</span>';
         } else {
             $page_links[] = sprintf(
-                "<a class='next-page' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
+                "<a class='next-page button' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
                 esc_url( add_query_arg( 'paged', min( $total_pages, $current + 1 ), $current_url ) ),
                 __( 'Next page' ),
@@ -865 +865 @@
 
         if ( $disable_last ) {
-            $page_links[] = '<span class="tablenav-pages-navspan" aria-hidden="true">&raquo;</span>';
+            $page_links[] = '<span class="tablenav-pages-navspan button disabled" aria-hidden="true">&raquo;</span>';
         } else {
             $page_links[] = sprintf(
-                "<a class='last-page' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
+                "<a class='last-page button' href='%s'><span class='screen-reader-text'>%s</span><span aria-hidden='true'>%s</span></a>",
                 esc_url( add_query_arg( 'paged', $total_pages, $current_url ) ),
                 __( 'Last page' ),