Make WordPress Core

Opened 10 years ago

Closed 5 years ago

#32679 closed defect (bug) (invalid)

Widgets inside customiser lack contrast

Reported by: paulwilde's profile paulwilde Owned by:
Milestone: Priority: normal
Severity: normal Version: 4.3
Component: Customize Keywords: needs-patch
Focuses: ui Cc:

Description

Ever since #31336 was introduced the contract of the widget borders has gotten worst.

The border colours for the customiser interface seem to all be #ddd, whereas the border colour for the widgets are currently set as #e5e5e5. My suggestion would be to use #ddd.

#e5e5e5 is used inside the admin where the widgets (and menus) are on top of a white background, however the background has been changed to #eee so the edges of the widgets now look blurry and a little foreign from the rest of the interface.

See my attached screenshot of its current state and then what I would suggest.

Also for reference #29158.

Attachments (1)

widget-borders.png (96.2 KB) - added by paulwilde 10 years ago.

Download all attachments as: .zip

Change History (10)

This ticket was mentioned in Slack in #design by ocean90. View the logs.


10 years ago

#2 @celloexpressions
10 years ago

Left a bunch of comments in slack, but we never redesigned these in #31336. The current proposed design is what was merged with Menu Customizer, once we finalize that we'll want to make widgets match.

#3 @designsimply
10 years ago

#32727 was marked as a duplicate.

#4 @celloexpressions
10 years ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 4.3
  • Type changed from enhancement to defect (bug)

Waiting for a decision on #29158 to patch this, but we need to make widgets and menus match. Currently leaning towards doing white backgrounds for titles, dark borders on hover and expanded, and an in between gray shared by the heading and the form in expanded controls. The titles definitely need to be on a white background when they aren't expanded for visual contrast reasons (especially noticeable with menus). Would like to look at darkening border colors on inputs universally as well.

#5 @obenland
10 years ago

  • Milestone changed from 4.3 to Future Release

#29158 got moved out of the milestone, let's look at them together in a future release.

#6 follow-up: @celloexpressions
9 years ago

Widget styling needs to be updated to match the newer menus UI, where contrast decisions were carefully considered around the principles established in #31336 (primarily using white backgrounds for interaction elements). This is largely independent of #29158 at this point, although I'm going to cover the available widgets panel in that patch.

#9 in reply to: ↑ 6 @celloexpressions
6 years ago

  • Milestone set to Future Release

My latest comment still stands. This needs a patch accordingly.

Replying to celloexpressions:

Widget styling needs to be updated to match the newer menus UI, where contrast decisions were carefully considered around the principles established in #31336 (primarily using white backgrounds for interaction elements). This is largely independent of #29158 at this point, although I'm going to cover the available widgets panel in that patch.

#10 @dlh
5 years ago

@paulwilde @celloexpressions Is this ticket still needed following the admin CSS changes in WordPress 5.3?

#11 @dlh
5 years ago

  • Milestone Future Release deleted
  • Resolution set to invalid
  • Status changed from new to closed

As far as I can tell, the borders referenced here that were #e5e5e5 are now #ccd0d4 after the admin CSS changes in 5.3. Accordingly, I'm thinking that the context in which this ticket was written has been superseded, and that any subsequent discussions about the widgets in the Customizer would get better feedback in a new ticket referencing the admin CSS tickets. If I'm wrong about this, please feel free to reopen.

Note: See TracTickets for help on using tickets.