Ticket #23120: 23120.diff

File 23120.diff, 2.2 KB (added by cdog, 4 months ago)
Line 
1Index: wp-admin/js/widgets.js
2===================================================================
3--- wp-admin/js/widgets.js      (revision 23299)
4+++ wp-admin/js/widgets.js      (working copy)
5@@ -175,8 +175,10 @@
6        },
7 
8        saveOrder : function(sb) {
9+               var spinner;
10+
11                if ( sb )
12-                       $('#' + sb).closest('div.widgets-holder-wrap').find('.spinner').css('display', 'inline-block');
13+                       spinner = $('#' + sb).closest('div.widgets-holder-wrap').find('.sidebar-name .spinner').css('display', 'inline-block');
14 
15                var a = {
16                        action: 'widgets-order',
17@@ -190,7 +192,18 @@
18                });
19 
20                $.post( ajaxurl, a, function() {
21-                       $('.spinner').hide();
22+                       if ( spinner ) {
23+                               spinner.addClass('success').fadeOut('slow', function() {
24+                                       $(this).removeClass('success');
25+                               });
26+                       }
27+               })
28+               .error( function() {
29+                       if ( spinner ) {
30+                               spinner.addClass('error').fadeOut('slow', function() {
31+                                       $(this).removeClass('error');
32+                               });
33+                       }
34                });
35 
36                this.resize();
37@@ -235,7 +248,10 @@
38                                        wpWidgets.resize();
39                                }
40                        } else {
41-                               $('.spinner').hide();
42+                               $('.spinner', widget).addClass('success').fadeOut('slow', function() {
43+                                       $(this).removeClass('success');
44+                               });
45+
46                                if ( r && r.length > 2 ) {
47                                        $('div.widget-content', widget).html(r);
48                                        wpWidgets.appendTitle(widget);
49@@ -244,6 +260,11 @@
50                        }
51                        if ( order )
52                                wpWidgets.saveOrder();
53+               })
54+               .error( function() {
55+                       $('.spinner', widget).addClass('error').fadeOut('slow', function() {
56+                               $(this).removeClass('error');
57+                       });
58                });
59        },
60 
61Index: wp-admin/css/wp-admin.css
62===================================================================
63--- wp-admin/css/wp-admin.css   (revision 23299)
64+++ wp-admin/css/wp-admin.css   (working copy)
65@@ -4190,6 +4190,14 @@
66        margin: 5px 5px 0;
67 }
68 
69+.spinner.success {
70+       background-image: url(../images/yes.png);
71+}
72+
73+.spinner.error {
74+       background-image: url(../images/no.png);
75+}
76+
77 .no-float {
78        float: none;
79 }
80@@ -8512,6 +8520,14 @@
81        .customize-loading #customize-container {
82                background-image: url(../images/wpspin_light-2x.gif);
83        }
84+
85+       .spinner.success {
86+               background-image: url(../images/yes-2x.png);
87+       }
88+
89+       .spinner.error {
90+               background-image: url(../images/no-2x.png);
91+       }
92 }
93 
94 /* =Localized CSS