Make WordPress Core

Opened 4 years ago

Last modified 4 years ago

#51522 new defect (bug)

Customizing : Site Icon CSS Change

Reported by: ravipatel's profile ravipatel Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Customize Keywords: has-patch needs-refresh 2nd-opinion
Focuses: css Cc:

Description

When image not exits on provided path display css issue.
Please review bellow screenshots.

Issue :

https://prnt.sc/uzb75e

Solution :

https://prnt.sc/uzb622

Attachments (1)

51522-site-icon.css.patch (382 bytes) - added by ravipatel 4 years ago.
css changes for a site icon

Download all attachments as: .zip

Change History (4)

@ravipatel
4 years ago

css changes for a site icon

#1 @dlh
4 years ago

  • Component changed from Administration to Customize
  • Focuses ui removed
  • Keywords reporter-feedback added
  • Version trunk deleted

Hi @ravipatel,

I'm having difficulty understanding the issue from the screenshots. Could you provide steps to replicate the issue?

#2 @ravipatel
4 years ago

1) Add favicon icon - I am using png image

2) Remove from upload directory or rename image name temp for show wrong path

3) After image path not found you can see this ui issue.

https://prnt.sc/uzb75e

#3 @sabernhardt
4 years ago

  • Keywords needs-refresh 2nd-opinion added; reporter-feedback removed

"Remove from upload directory or rename image" is the key step to reproduce the missing image error. When the site icon image is deleted in the media library, the site icon setting is removed and the Customizer no longer tries to show a preview.

To adjust the alt text styles, setting display: inline-block and changing the color works here. Adjusting the padding is not a good option, though, because it would change the app icon preview when the image exists at the expected URL.

Version 0, edited 4 years ago by sabernhardt (next)
Note: See TracTickets for help on using tickets.