[wp-trac] [WordPress Trac] #16434: Give site admin ability to upload favicon in Settings, General
WordPress Trac
wp-trac at lists.automattic.com
Mon Feb 27 17:48:34 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 nacin):
Replying to [comment:88 TomAuger]:
> Would we consider releasing the next version with a ''default''
WordPress favicon in place (possibly in the root directory) in .png and
.ico format? A fallback for when you have not defined one yourself in
Settings > General? This would also help with a lot of 404's that servers
are getting.
No, a few reasons: One, it is difficult to then go ahead and remove that
file (otherwise, some browsers will still read it). Two, there's really no
reason for a default WordPress favicon on every WP.org site out there.
Three, it only works for root-level sites. Four, we already intercept
root-level /favicon.ico requests early and return a zero-length
image/vnd.microsoft.com response, so 404s are not going to end up being
logged if external rewrite rules are in place to map everything to
index.php. Some hosts have simply placed favicon.ico files in root
directories to even prevent WordPress from loading in these cases, but I
don't think that's our responsibility beyond what we already do.
I agree that supporting ICO should be the goal. Let's just not catch
ourselves up over it. Still need to build a working feature. :-)
--
Ticket URL: <http://core.trac.wordpress.org/ticket/16434#comment:89>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list