[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:52:31 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 chipbennett):

 Replying to [comment:80 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.

 +1 to integrating the WordPress Filesystem API into the library.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/16434#comment:81>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list