Changeset 35854 for trunk/src/wp-admin/includes/class-wp-upgrader-skins.php

Timestamp:

12/10/2015 05:20:18 PM (9 years ago)

Author:

afercia

Message:

Accessibility: the iframes used for the update progress in the Updates screen need title attributes to describe their content.

Fixes #34763.

File:

• trunk/src/wp-admin/includes/class-wp-upgrader-skins.php (modified) (1 diff)

trunk/src/wp-admin/includes/class-wp-upgrader-skins.php

@@ -225 +225 @@
         $this->plugin = $this->upgrader->plugin_info();
         if ( !empty($this->plugin) && !is_wp_error($this->result) && $this->plugin_active ){
-            echo '<iframe style="border:0;overflow:hidden" width="100%" height="170px" src="' . wp_nonce_url('update.php?action=activate-plugin&networkwide=' . $this->plugin_network_active . '&plugin=' . urlencode( $this->plugin ), 'activate-plugin_' . $this->plugin) .'"></iframe>';
+            // Currently used only when JS is off for a single plugin update?
+            echo '<iframe title="' . esc_attr( 'Update progress' ) . '" style="border:0;overflow:hidden" width="100%" height="170" src="' . wp_nonce_url( 'update.php?action=activate-plugin&networkwide=' . $this->plugin_network_active . '&plugin=' . urlencode( $this->plugin ), 'activate-plugin_' . $this->plugin ) . '"></iframe>';
         }