Changeset 29031 for trunk/src/wp-admin/theme-install.php

Timestamp:

07/08/2014 06:18:23 PM (12 years ago)

Author:

helen

Message:

More consistent icons in place of some buttons for the Customizer and theme install preview.

• Use a single close icon instead of a context-changing cancel/close button in the Customizer. In user testing, this small distinction in language was overlooked, and we use an AYS now instead. This also makes it consistent with the back arrow in Customizer panels.
• Use the same close icon as well as the previous/next arrows from the theme details browser in the theme install preview.

props celloexpressions. fixes #28655.

File:

• trunk/src/wp-admin/theme-install.php (modified) (2 diffs)

trunk/src/wp-admin/theme-install.php

@@ -226 +226 @@
     <div class="wp-full-overlay-sidebar">
         <div class="wp-full-overlay-header">
-            <a href="#" class="close-full-overlay button-secondary"><?php _e( 'Close' ); ?></a>
+            <a href="#" class="close-full-overlay"><span class="screen-reader-text"><?php _e( 'Close' ); ?></span></a>
+            <a href="#" class="previous-theme"><span class="screen-reader-text"><?php _ex( 'Previous', 'Button label for a theme' ); ?></span></a>
+            <a href="#" class="next-theme"><span class="screen-reader-text"><?php _ex( 'Next', 'Button label for a theme' ); ?></span></a>
         <# if ( data.installed ) { #>
             <a href="#" class="button button-primary theme-install disabled"><?php _ex( 'Installed', 'theme' ); ?></a>
@@ -263 +265 @@
                 <span class="collapse-sidebar-arrow"></span>
             </a>
-            <div class="theme-navigation">
-                <a class="previous-theme button" href="#"><?php _ex( 'Previous', 'Button label for a theme' ); ?></a>
-                <a class="next-theme button" href="#"><?php _ex( 'Next', 'Button label for a theme' ); ?></a>
-            </div>
         </div>
     </div>