Make WordPress Core

Opened 8 years ago

Closed 7 years ago

#38010 closed defect (bug) (fixed)

Minor fixes: Media, Add Plugins, and Add Themes top toolbar inconsistencies

Reported by: afercia's profile afercia Owned by: afercia's profile afercia
Milestone: 4.7 Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-screenshots good-first-bug has-patch
Focuses: ui Cc:

Description

A few, minor, visual inconsistencies in these toolbars that would be nice to fix, see also the screenshot below:

  • the right margin is different
  • placeholders: title case or not title case?
  • placeholders: ellipsis or not ellipsis?

About the ellipsis: it would be nice to reach some consensus and standardise across all the admin. I'd say remove them because they're useless and can be problematic for translations.

Side note: for accessibility, placeholders should not be used as replacement for labels but this should be discussed in a separate ticket.

Worth noting also the input fields size is a bit different, but just under a certain viewport size, I guess this is not a big deal.

https://cldup.com/rl8x5f3HpI.png

Attachments (7)

38010.patch (862 bytes) - added by mikeviele 7 years ago.
Fixes Title casing of "Search Themes..." placeholder
fixing-padding-and-ellipsis.patch (2.9 KB) - added by jamesacero 7 years ago.
Fixing search bar padding to match on all pages, removed ellipsis from placeholder text on all search bars
38010-pad-fix-and-theme-placeholders.patch (1.9 KB) - added by mikeviele 7 years ago.
screenshot-add-plugins.png (26.4 KB) - added by koenschipper 7 years ago.
screenshot-add-themes.png (25.0 KB) - added by koenschipper 7 years ago.
screenshot-media-library.png (22.6 KB) - added by koenschipper 7 years ago.
38010.diff (3.3 KB) - added by afercia 7 years ago.

Download all attachments as: .zip

Change History (16)

@mikeviele
7 years ago

Fixes Title casing of "Search Themes..." placeholder

@jamesacero
7 years ago

Fixing search bar padding to match on all pages, removed ellipsis from placeholder text on all search bars

#1 @mikeviele
7 years ago

  • Keywords has-patch added

The ellipsis was not present in the current build as of the patch submission above (https://core.trac.wordpress.org/attachment/ticket/38010/38010.patch), but instead was converted to three periods (Search Themes...) with a comment stating

// placeholder (no ellipsis)

I opted to leave the the three periods in place.

Last edited 7 years ago by mikeviele (previous) (diff)

#2 follow-up: @mattking5000
7 years ago

for the sake of consistency, shouldn't the media library placeholder text be "Search Media" ?

#3 in reply to: ↑ 2 @mikeviele
7 years ago

Replying to mattking5000:

for the sake of consistency, shouldn't the media library placeholder text be "Search Media" ?

I agree that would be more consistent Matt. Not sure about the ellipses though. Current WP build uses 3 periods to "Search Themes..." so someone has looked at it. I'm not sure if we use "..." consistently across all placeholders like these or not.

#4 @afercia
7 years ago

For more details about the ellipses issue, see #32875. Basically, in some places there's the need to use three periods, so the inline comment // placeholder (no ellipsis) means don't use HTML entities or UTF-8 characters.

Changes to wp-admin/includes/class-walker-nav-menu-edit.php should be removed from the patch.

P.S.
The left padding should be preserved.

Last edited 7 years ago by afercia (previous) (diff)

#5 @mikeviele
7 years ago

Thanks @afercia. I've updated with a new patch combining mine and @jamesacero 's work above.

#6 @koenschipper
7 years ago

I've tested it in Chrome and Safari. It all looks good. Contributor Day Wordcamp Netherlands :)

#7 @dungengronovius
7 years ago

Same result in Firefox, looks as expected

@afercia
7 years ago

#8 @afercia
7 years ago

  • Milestone changed from Awaiting Review to 4.7

Checking all the currently used placeholders (including on the Customizer) in most of the cases they don't use title case and use the three dots. Let's standardise on this.

The refreshed patch also expands the Media search placeholder in Search media items... for consistency with the Plugins and Themes toolbars.

#9 @afercia
7 years ago

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

In 38795:

Administration: Better consistency for the Media, Add Plugins, and Add Themes toolbars.

Fixes the Add Themes toolbar padding. Standardizes the placeholders to no title
case and three trailing dots. Expands the Media search placeholder in
"Search media items..." for consistency with the Plugins and Themes toolbars.

Props mikeviele, jamesacero, mattking5000, koenschipper, dungengronovius.

Fixes #38010.

Note: See TracTickets for help on using tickets.