Make WordPress Core

Opened 10 years ago

Closed 10 years ago

Last modified 8 years ago

#26496 closed defect (bug) (fixed)

Themes: "Search" is ambiguous and confusing

Reported by: nacin's profile nacin Owned by: nacin's profile nacin
Milestone: 3.8 Priority: high
Severity: major Version: 3.8
Component: Customize Keywords: has-patch commit
Focuses: Cc:


Here's what you see on the themes page:

Themes (#) [Add New] [Search...]

If you swapped the search input for Add New, you'd think it was a search input for new themes. I think that it's highly likely that even in their current position, that is exactly what a lot of people may do.

We've had this confusion with both plugin and theme search fields (hell, I've done it myself), which is why the labels for the search submit buttons are "Search Installed Themes" and "Search Installed Plugins".

The search is also missing a label (even just screen-reader-text). We had this in 3.7:

<label class="screen-reader-text" for="theme-search-input"><?php _e('Search Installed Themes'); ?>:</label>


  • Add missing label, as above
  • Change placeholder to be "Search installed themes..."
  • Try to float the search input to the right, as it is on every other screen in the admin

Attachments (4)

26496.patch (1.5 KB) - added by SergeyBiryukov 10 years ago.
26496.after.png (654.9 KB) - added by SergeyBiryukov 10 years ago.
26496.diff (1.4 KB) - added by nacin 10 years ago.
26496.2.diff (1.4 KB) - added by nacin 10 years ago.

Download all attachments as: .zip

Change History (10)

#1 @SergeyBiryukov
10 years ago

  • Keywords has-patch added

10 years ago

10 years ago

#2 @nacin
10 years ago

  • Keywords commit added

26496.2.diff shares the existing string. The 3.7 label had the colon outside the translation.

#3 @nacin
10 years ago

Some chatter here: Floating it right would be nice but it messes with the design too much. The label and placeholder are fine. We also don't have a live-search on any other screen (closest to it is internal linking) so the UX is different anyway.

#4 @nacin
10 years ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In 26827:

Themes: use explicit 'search installed themes' label and placeholder.

fixes #26496.

#5 @matveb
9 years ago

What about "Search your themes" instead?

This ticket was mentioned in Slack in #accessibility by afercia. View the logs.

8 years ago

Note: See TracTickets for help on using tickets.