[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:
 Severity:  normal                               |  Resolution:
 Keywords:  ux-feedback has-patch needs-testing  |     Focuses:  ui
  needs-unit-tests has-screenshots               |
-------------------------------------------------+-------------------------

Comment (by michaelarestad):

 We need to answer some questions before jumping into implementing anything
 like Jetpack.

 1. What problem is this solving?
 2. Who is this for?
 3. Does this solution need to be global? (stays set no matter which theme
 is active regardless of theme support)

 My opinion:

 This should be a theme-specific setting. The theme has a better idea of
 what size restrictions there are and can possibly help with positioning.
 Why not let the customizer do it's thing? Can't a theme add a site logo
 image form of some sort?

 More importantly, treating it as a standard image addition tool for themes
 means the theme can add other image/logo areas that aren't separate from
 the global one in the UI. I've used themes that require a full logo and a
 small logo (for collapsed fix nav). It would be really frustrating if I
 could set the full one in one spot and had to hunt for the other.

 If it becomes a global, I think it will ultimately cause more confusion
 and struggle for the user.

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