Changeset 39528

Timestamp:

12/06/2016 11:11:08 PM (8 years ago)

Author:

afercia

Message:

Accessibility: Remove inappropriate content from the Themes screen heading.

Props joedolson.

See #26601.

Location:

trunk/src/wp-admin

Files:

• css/themes.css (modified) (1 diff)
• js/theme.js (modified) (2 diffs)
• themes.php (modified) (1 diff)

trunk/src/wp-admin/css/themes.css

@@ -950 +950 @@
         clear: both;
         left: 0;
-        top: 0;
         right: 0;
-        margin: 10px 0;
+        margin: -5px 0 20px 0;
         width: 100%;
         max-width: 280px;

trunk/src/wp-admin/js/theme.js

@@ -84 +84 @@
 
     // Defines search element container
-    searchContainer: $( '#wpbody h1:first' ),
+    searchContainer: $( '.search-form' ),
 
     // Search input and view
@@ -1774 +1774 @@
     activeClass: 'current',
 
-    // Overwrite search container class to append search
-    // in new location
-    searchContainer: $( '.wp-filter .search-form' ),
-
     /*
      * When users press the "Upload Theme" button, show the upload form in place.

trunk/src/wp-admin/themes.php

@@ -153 +153 @@
 
 <div class="wrap">
-    <h1><?php esc_html_e( 'Themes' ); ?>
+    <h1 class="wp-heading-inline"><?php esc_html_e( 'Themes' ); ?>
         <span class="title-count theme-count"><?php echo count( $themes ); ?></span>
+    </h1>
+
     <?php if ( ! is_multisite() && current_user_can( 'install_themes' ) ) : ?>
         <a href="<?php echo admin_url( 'theme-install.php' ); ?>" class="hide-if-no-js page-title-action"><?php echo esc_html_x( 'Add New', 'Add new theme' ); ?></a>
     <?php endif; ?>
-    </h1>
+
+    <span class="search-form"></span>
+
+    <hr class="wp-header-end">
 <?php
 if ( ! validate_current_theme() || isset( $_GET['broken'] ) ) : ?>