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

[WordPress Trac]

#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 has-patch  |
--------------------------------+------------------------------

Comment (by jacopo.vip):

 Replying to [comment:4 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 - it
 now also shows the checkmark briefly.
 >
 > tested on mac chrome and firefox


 I think that's a perfect solution. Good work :)

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