[wp-trac] [WordPress Trac] #16434: Give site admin ability to upload favicon in Settings, General

[WordPress Trac]

#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  |
-------------------------------------+------------------
Changes (by TomAuger):

 * keywords:   => has-patch needs-testing


Comment:

 So here's what we have so far:
 - Rudimentary favicon image upload under Settings > General.
 - Crop and save out a .PNG and .ICO file (.ico file format is currently
 incorrect and does NOT work in IE)
 - Original upload AND both thumbs are stored as attachments
 - Hooks into wp_head and admin_head to add the x-browser HTML that (once
 the ICO format is fixed) works

 Immediate TODOs before we could even consider releasing this:
 - clean up the Settings UI and display the uploaded thumb
 - Add the Remove feature
 - fix the ICO format or provide a workaround for IE

 Later:
 - encapsulate in a class
 - some error trapping was skipped (marked in TODOs)
 - slicker upload UI (at least have javascript click the "upload" button
 once you've selected a file!
 - admin notice or some other method to "close the circle" and let the
 admin know the favicon was uploaded successfully.

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