[wp-trac] [WordPress Trac] #16434: Give site admin ability to upload favicon in Settings, General

[WordPress Trac]

#16434: Give site admin ability to upload favicon in Settings, General
--------------------------+------------------------------------------------
 Reporter:  jane          |       Owner:  obenland
     Type:  task          |      Status:  reopened
  (blessed)               |   Milestone:  4.3
 Priority:  normal        |     Version:  3.1
Component:  Customize     |  Resolution:
 Severity:  normal        |     Focuses:  ui, accessibility, administration
 Keywords:  has-patch     |
--------------------------+------------------------------------------------

Comment (by celloexpressions):

 I'm also +1 for Customizer-only, because we've been spending a lot of
 effort trying to remove options from (and reduce the number of) settings
 screens, and encouraging users to do this as part of the Customizer flow
 is a good step towards the future.

 Re: Drew's comments, I'd like to see the browser chrome image at least be
 more generic so that it can convey the meaning without being a specific
 identifiable browser and device. The image should be grayscale at the
 least, as the colored buttons are distracting, and not something you find
 on Windows, for example. We should remove the default button - I kind of
 forgot that was there. I don't think we're setting a default value, so
 that should just be "remove". Should be able to do that in the Site Icon
 control or with CSS without overriding the entire `content_template()`,
 hopefully. Or even better, in `WP_Customize_Media_Control`, if the default
 is empty, show the remove button instead of a default button.

 Great work bringing this together in the media modal @obenland!

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