[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 ux-feedback  |
-------------------------------------------------+------------------

Comment (by azaozz):

 Few more things while testing 16434.2.diff:

 - When uploading a "favicon.ico" file, it's uploaded but is not set as
 favicon. Instead there's the error: "Please only use PNG (.png), JPEG
 (.jpg) or BMP (.bmp) image files for favicons." (this statement is wrong
 by itself).

 - When uploading an already resized 32x32 image it still goes to the crop
 screen, would probably be better to skip that step and set the image
 directly as favicon.

 - The UI on the General Settings screen should probably be after "E-mail
 Address" or at the bottom. Having it at the top doesn't "feel right".

 Also using thickbox wouldn't be advisable. There are plans for scrapping
 thickbox and modal dialogs in general. Perhaps better would be to insert
 an iframe right under the UI on the General Settings screen.

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