Changeset 29201 for trunk/src/wp-admin/includes/class-wp-plugin-install-list-table.php

Timestamp:

07/16/2014 10:49:53 PM (10 years ago)

Author:

wonderboymusic

Message:

In Plugin Install List Table, remove title attribute for plugin action links; add screen-reader-text.

Props joedolson.
Fixes #28853.

File:

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

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

@@ -260 +260 @@
                     case 'install':
                         if ( $status['url'] ) {
-                            $action_links[] = '<a class="install-now button" href="' . $status['url'] . '" title="' . esc_attr( sprintf( __( 'Install %s' ), $name ) ) . '">' . __( 'Install Now' ) . '</a>';
+                            $action_links[] = '<a class="install-now button" href="' . $status['url'] . '" aria-labelledby="' . $plugin['slug'] . '">' . __( 'Install Now' ) . '</a>';
                         }
 
@@ -266 +266 @@
                     case 'update_available':
                         if ( $status['url'] ) {
-                            $action_links[] = '<a class="button" href="' . $status['url'] . '" title="' . esc_attr( sprintf( __( 'Update to version %s' ), $status['version'] ) ) . '">' . __( 'Update Now' ) . '</a>';
+                            $action_links[] = '<a class="button" href="' . $status['url'] . '" aria-labelledby="' . $plugin['slug'] . '">' . __( 'Update Now' ) . '</a>';
                         }
 
@@ -280 +280 @@
                                 '&amp;TB_iframe=true&amp;width=600&amp;height=550' );
 
-            $action_links[] = '<a href="' . esc_attr( $details_link ) . '" class="thickbox" title="' .
-                                esc_attr( sprintf( __( 'More information about %s' ), $name ) ) . '">' . __( 'More Details' ) . '</a>';
+            $action_links[] = '<a href="' . esc_attr( $details_link ) . '" class="thickbox" aria-labelledby="' . $plugin['slug'] . '" data-title="' . esc_attr( $name ) . '">' . __( 'More Details' ) . '</a>';
+
 
             /**