WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 4 years ago

#26496 closed defect (bug) (fixed)

Themes: "Search" is ambiguous and confusing

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

Description

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>

Suggestions:

  • 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 6 years ago.
26496.after.png (654.9 KB) - added by SergeyBiryukov 6 years ago.
26496.diff (1.4 KB) - added by nacin 6 years ago.
26496.2.diff (1.4 KB) - added by nacin 6 years ago.

Download all attachments as: .zip

Change History (10)

#1 @SergeyBiryukov
6 years ago

  • Keywords has-patch added

@nacin
6 years ago

@nacin
6 years ago

#2 @nacin
6 years ago

  • Keywords commit added

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

#3 @nacin
6 years ago

Some chatter here: http://irclogs.wordpress.org/chanlog.php?channel=wordpress-dev&day=2013-12-09&sort=asc#m749264. 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
6 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
6 years ago

What about "Search your themes" instead?

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


4 years ago

Note: See TracTickets for help on using tickets.