[wp-trac] [WordPress Trac] #16434: Give site admin ability to upload favicon in Settings, General
WordPress Trac
wp-trac at lists.automattic.com
Fri Feb 10 16:38:52 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: |
-----------------------------+------------------
Comment (by TomAuger):
Replying to [comment:42 Otto42]:
> - Alpha channel support allows for transparency, which looks better in
the browser because you can let the background color through.
Because we're letting the user upload any image format and doing the
resize/crop, it's unlikely we'll want to support transparency. Unless we
have some kind of PNG pass through option, whereby if they happen to
upload a 24bit PNG, they get the option to save the raw (downscaled) PNG
in addition to the ICO. That seems like an enhancement rather than a first
release feature though.
> - You can use any size instead of being limited to 32x32 or 16x16.
I"m not sure I see how this works - after all, the browser chrome doesn't
allow for a larger icon, at least not in any browser I'm familiar with.
IE, Chrome, FF etc all put the favicon inside the address bar, and that
ain't going to grow.
> - PNGs are easily supported. Writing ICOs is kind of a pain.
Yes, but with the lib you point to in an earlier comment this should be
moot.
> PNGs are more standard. ICOs are weird. It's not hard to do both if this
is done.
We could certainly consider saving out both versions and the serving up
the ICO based on conditional tags. I suppose the process should be:
1. upload
2. crop if necessary
3. resize if necessasry
4. save out to PNG with imageMagick
5. convert to ICO
--
Ticket URL: <http://core.trac.wordpress.org/ticket/16434#comment:43>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list