[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