Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#35826 closed enhancement (fixed)

Customizer: Contain "No Image Set" in dashed border

Reported by: melchoyce Owned by: ocean90
Priority: normal Milestone: 4.5
Component: Customize Version: 3.4
Severity: normal Keywords: good-first-bug has-patch
Cc: Focuses: ui

Description

In the Customizer, in panels like Site Identity, Header Image, and Background Image, the "no image set" label seems to just float there. We should add a dashed border to that container div so it looks grounded.

See attached images for before/after.

Attachments (5)

no-image-set-before.png (492.8 KB ) - added by melchoyce 11 years ago.
no-image-set-after.png (471.6 KB ) - added by melchoyce 11 years ago.
35826.patch (516 bytes ) - added by Ankit K Gupta 11 years ago.
added patch
35826.1.patch (451 bytes ) - added by Ankit K Gupta 11 years ago.
Updated patch file
35826.2.patch (642 bytes ) - added by chris_dev 11 years ago.
"No image selected" is not vertically aligned because the border 2 pixels so I changed "top: 10px" to "8px"

Download all attachments as: .zip

Change History (11)

#1 @westonruter
11 years ago

  • Keywords needs-patch good-first-bug added
  • Milestone Awaiting Review4.5

@Ankit K Gupta
11 years ago

added patch

#2 @Ankit K Gupta
11 years ago

  • Keywords has-patch added; needs-patch removed

#3 follow-up: @westonruter
11 years ago

@ankit-k-gupta thank you for the patch. I think, however, that the CSS selector should be a bit more specific, scoped by the control's class name perhaps?

Last edited 11 years ago by westonruter (previous) (diff)

#4 in reply to: ↑ 3 @Ankit K Gupta
11 years ago

Replying to westonruter:

@ankit-k-gupta thank you for the patch. I think, however, that the CSS selector should be a bit more specific, scoped by the control's class name perhaps?

@westonruter Thanks for your feedback, I am not expert in CSS, hope this fulfill the requirement for this bug.
Added updated patch.
Thanks :)

@Ankit K Gupta
11 years ago

Updated patch file

@chris_dev
11 years ago

"No image selected" is not vertically aligned because the border 2 pixels so I changed "top: 10px" to "8px"

#5 @ocean90
11 years ago

  • Owner set to ocean90
  • Resolutionfixed
  • Status newclosed

In 36639:

Customizer: Contain "No image set/selected" in dashed border.

Props Ankit K Gupta, chris_dev.
Fixes #35826.

#6 @johnbillion
11 years ago

  • Version trunk3.4
Note: See TracTickets for help on using tickets.