Ticket #25060: 25060.diff

src/wp-admin/css/list-tables.css

@@ -213 +213 @@
     border-top: 1px solid rgba(0, 0, 0, 0.03);
 }
 
+/* Green used in JS to indicate an event is successfully happening - important to override other specificity */
+.in-transition {
+        background-color: #ceb !important;
+}
+
 /* table vim shortcuts */
 .vim-current,
 .vim-current th,

src/wp-admin/js/edit-comments.js

@@ -83 +83 @@
 
                         a.click(function(){
                                 list.wpList.del(this);
-                                $('#undo-' + id).css( {backgroundColor:'#ceb'} ).fadeOut(350, function(){
+                                $('#undo-' + id).addClass('in-transition').fadeOut(350, function(){
                                         $(this).remove();
-                                        $('#comment-' + id).css('backgroundColor', '').fadeIn(300, function(){ $(this).show(); });
+                                        $('#comment-' + id).fadeIn(300, function(){ $(this).show(); });
                                 });
                                 return false;
                         });
@@ -338 +338 @@
 
                 if ( this.cid && this.act == 'edit-comment' ) {
                         c = $('#comment-' + this.cid);
-                        c.fadeIn(300, function(){ c.show(); }).css('backgroundColor', '');
+                        c.fadeIn(300, function(){ c.show(); });
                 }
 
                 // reset the Quicktags buttons
@@ -469 +469 @@
         },
 
         show : function(xml) {
-                var t = this, r, c, id, bg, pid;
+                var t = this, r, c, id, pid;
 
                 if ( typeof(xml) == 'string' ) {
                         t.error({'responseText': xml});
@@ -495 +495 @@
                         updatePending( -1 );
 
                         if ( this.comments_listing == 'moderated' ) {
-                                pid.animate( { 'backgroundColor':'#CCEEBB' }, 400, function(){
-                                        pid.fadeOut();
-                                });
+                                pid.addClass('in-transition', 400).fadeOut();
                                 return;
                         }
                 }
@@ -508 +506 @@
 
                 id = $(id);
                 t.addEvents(id);
-                bg = id.hasClass('unapproved') ? '#FFFFE0' : id.closest('.widefat, .postbox').css('backgroundColor');
 
-                id.animate( { 'backgroundColor':'#CCEEBB' }, 300 )
-                        .animate( { 'backgroundColor': bg }, 300, function() {
-                                if ( pid && pid.length ) {
-                                        pid.animate( { 'backgroundColor':'#CCEEBB' }, 300 )
-                                                .animate( { 'backgroundColor': bg }, 300 )
-                                                .removeClass('unapproved').addClass('approved')
-                                                .find('div.comment_status').html('1');
-                                }
-                        });
+                id.addClass('in-transition', 300).removeClass('in-transition', 300);
 
+                if ( pid && pid.length ) {
+                        pid.addClass('in-transition', 300).removeClass('in-transition', 300).removeClass('unapproved').addClass('approved').find('div.comment_status').html('1');
+                }
         },
 
         error : function(r) {