[wp-trac] [WordPress Trac] #33755: Add Site Logo to WordPress Core

[WordPress Trac]

#33755: Add Site Logo to WordPress Core
-------------------------------------------------+-------------------------
 Reporter:  fatmedia                             |       Owner:
     Type:  feature request                      |      Status:  reopened
 Priority:  normal                               |   Milestone:  4.5
Component:  Customize                            |     Version:  trunk
 Severity:  normal                               |  Resolution:
 Keywords:  ux-feedback has-patch needs-testing  |     Focuses:  ui
  needs-unit-tests has-screenshots               |
-------------------------------------------------+-------------------------

Comment (by mark-k):

 Replying to [comment:90 benoitchantre]:
 > I think that items localed in the Site Identity are global things that
 can be used both on front-end and back-end.
 > What do you think of the idea to use the site logo in wp-login.php
 (#35807)?

 The important thing about images is to keep their aspect ratio. As long as
 you keep it and downsize or upsize by a reasonable factor nothing horrible
 will happen, but downsize too much and you just get a mesh of colors and
 upsize too much and you get pixelization. The problem with the site logo
 as a global resource is that there are no standards for the logo size and
 if theme A have only 100 pixels available while theme B has 200 you will
 either end up with pixelization if you upsize or some ugly empty space if
 you don't. Switching from a logo optimized for B to A will result in a
 logo which is not displayed to the optimum especially if it contains text.

 Using the login logo as the site logo makes IMO more sense as everybody
 knows what is the relevant size and can plan for it. Down side is that it
 will "hardcode" the size of the logo in the login screen, but I don't
 remember that it had ever changed since the current design was introduced
 long time ago.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/33755#comment:91>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform