Ticket #23120: 23120.8.diff

Users/mepstein/Sites/elftest.dev/wp-admin/css/wp-admin.css

@@ -5938 +5938 @@
         margin: 2px 5px 0;
 }
 
+.spinner.success:before { 
+    font: normal 20px/1 'dashicons';
+    content: '\f147';
+    color: #e9ffe9;
+} 
+
+.spinner.error { 
+    font: normal 20px/1 'dashicons';
+    content: '\f335';
+    color: #ffe9e9; 
+} 
+
 .no-float {
         float: none;
 }

Users/mepstein/Sites/elftest.dev/wp-admin/js/widgets.js

@@ -313 +313 @@
                         action: 'widgets-order',
                         savewidgets: $('#_wpnonce_widgets').val(),
                         sidebars: []
-                };
+                }, spinner;
 
                 if ( sidebarId ) {
-                        $( '#' + sidebarId ).find('.spinner:first').css('display', 'inline-block');
+                        spinner = $( '#' + sidebarId ).find('.spinner:first').css('display', 'inline-block');
                 }
 
                 $('div.widgets-sortables').each( function() {
@@ -326 +326 @@
                 });
 
                 $.post( ajaxurl, data, function() {
-                        $('.spinner').hide();
+                        if ( spinner ) {
+                                spinner.addClass( 'success' ).css( 'opacity', 1 ).fadeOut( 'slow', function() {
+                                        spinner.removeClass( 'success' );
+                                });
+                        }
+                })
+                .error( function() {
+                        if ( spinner ) {
+                                spinner.addClass( 'error' ).css( 'opacity', 1 ).fadeOut( 'slow', function() {
+                                        spinner.removeClass( 'error' );
+                                });
+                        }
                 });
         },
 
@@ -372 +383 @@
                                         widget.remove();
                                 }
                         } else {
-                                $('.spinner').hide();
+                                $('.spinner', widget).addClass('success').fadeOut('slow', function() {
+                                    $(this).removeClass('success'); 
+                                });
                                 if ( r && r.length > 2 ) {
                                         $( 'div.widget-content', widget ).html(r);
                                         wpWidgets.appendTitle( widget );