[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