[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 16:50:50 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):
ImageICO library works great as advertised. Only concern is that it uses
fopen() and fwrite() to create the local file, and I wonder whether this
could be a problem.
If this ''is'' a concern, then there are two strategies:
1. edit/change the library to use more WP core methods
2. capture the ICO on stdout and save it that way.
Any thoughts? I could use some guidance on the best practice here.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/16434#comment:80>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list