[wp-trac] [WordPress Trac] #23120: There should be indication that widget settings have been saved

WordPress Trac noreply at wordpress.org
Fri Jan 4 21:41:27 UTC 2013


#23120: There should be indication that widget settings have been saved
-------------------------+------------------------------
 Reporter:  jacopo.vip   |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  Widgets      |     Version:  3.5
 Severity:  normal       |  Resolution:
 Keywords:  ui-focus     |
-------------------------+------------------------------

Comment (by adamsilverstein):

 i have attached a patch that i believe addresses this issue

 i added a class to wp-admin.css to show the yes.png checkmark bundled in
 /wp-admin/images

 i changed the hide() action in widgets.js when the save is complete,
 instead i add the new class and initiate a 1 second delay() with a half
 second fade at the end bringing opacity to 0. finally, a callback resets
 the opacity to 1 and completes the hide() in the original code.

 note! you can't just call delay(1500).hide() or the element gets hidden
 immediately (hide is not added to jquery's fx queue). the callback is
 required to have the hide actually happen after the delay. although the
 fade is not required i kinda think it looks nice.

 also, i changed the spinner for dragging and resorting the widgets sow it
 also shows the checkmark briefly.

 tested on mac chrome and firefox

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


More information about the wp-trac mailing list