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

WordPress Trac wp-trac at lists.automattic.com
Tue Feb 21 06:36:41 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  |
-------------------------------------+------------------

Comment (by jorbin):

 I did a little work building upon TomAuger's patch.  We are generating
 some notices, so eliminating those is going to be my next step.

 Note that the wp-facvicon.js file is just a copy of the .dev version so
 that you don't need to have SCRIPT_DEBUG set to true in order to use the
 patch.  Once we have something committable, we'll need to generate the
 proper file.

 UX wise:

 I think we should consider is having the upload take place in a lightbox.
 The experience of uploading a file and having that jump you to a new page
 seems wrong to me.

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


More information about the wp-trac mailing list