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

WordPress Trac wp-trac at lists.automattic.com
Mon Mar 5 19:33:13 UTC 2012


#16434: Give site admin ability to upload favicon in Settings, General
-------------------------------------------------+------------------
 Reporter:  jane                                 |       Owner:
     Type:  feature request                      |      Status:  new
 Priority:  normal                               |   Milestone:  3.4
Component:  Administration                       |     Version:  3.1
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch needs-testing ux-feedback  |
-------------------------------------------------+------------------

Comment (by TomAuger):

 Replying to [comment:111 azaozz]:
 > In general letting the user upload an image, then crop it and set it as
 a favicon implies expectations that it would work. However that won't work
 in 30%-90% of the browsers (IE), depending on the region. Also if the
 users are in IE, the favicon won't show even for them.

 The more I think about it, the more I feel strongly that releasing the
 favicon feature without IE / .ico support is just an out-and-out mistake.
 I think azaozz is right, support will be flooded, and the "build your own
 .ico and upload it" approach just seems too technical. If they can do
 that, why can't they just put the favicon.ico file in their site root and
 be done with it?

 Doesn't make sense.

 I think that releasing with some subset of ico2_3.php library, or possibly
 a homegrown version. All you need to do is play with encoding the
 bytestream, nothing too complicated or proprietary. Should be a matter of
 a couple hours to extract the relevant portions froom ico2_3, clean it up,
 add some WP error checking and best practices and roll it into favicon-
 upload.php.

 At the rate this is all going (I mean getting concensus, not the actual
 development), it looks doubtful for 3.4, unless we can all agree on the
 exact release spec.

 In the meanwhile, I'll work on cleaning up all the stuff that we DO have
 concensus on, and then see where we are in a day or two.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/16434#comment:117>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list