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

WordPress Trac noreply at wordpress.org
Sat Jan 5 15:56:54 UTC 2013


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

Comment (by adamsilverstein):

 Replying to [comment:9 SergeyBiryukov]:
 > [attachment:23120.patch]:
 > * Removes the class after hiding the element, otherwise the spinner
 image no longer appears on next click.
 > * When saving the order after dragging widgets, only displays the
 spinner in sidebar header (as it is now), not in each widget.
 > * Adds the widget object as a context to the selector, in order to only
 animate the spinner in the current widget.

 looks good. i had added the class removal on my end, but you beat me to
 it. plus your patch adds the widget context to the selection which makes
 sense, thanks.

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


More information about the wp-trac mailing list