Ticket #25060: 25060.4.patch

wp-admin/js/edit-comments.js

@@ -464 +464 @@
                 post.comments_listing = this.comments_listing;
                 post.p = $('[name="p"]').val();
 
-                if ( $('#comment-' + $('#comment_ID').val()).hasClass('unapproved') )
+                var commentID = $('#comment_ID').val();
+                var edit_row = $('#comment-' + commentID);
+                var remove_class;
+                if (edit_row.hasClass('unapproved'))
                         post.approve_parent = 1;
 
+                if (edit_row.prev().hasClass('even'))
+                        remove_class = true;
+
                 $.ajax({
                         type : 'POST',
                         url : ajaxurl,
                         data : post,
-                        success : function(x) { commentReply.show(x); },
+                        success : function(x) {
+                                commentReply.show(x);
+                                if (remove_class)
+                                        $('#comment-' + commentID).removeClass('even thread-even').addClass('alt odd thread-odd')
+                        },
                         error : function(r) { commentReply.error(r); }
                 });
 
@@ -479 +489 @@
         },
 
         show : function(xml) {
-                var t = this, r, c, id, bg, pid;
+                var t = this, r, c, id, bgClass, pid;
 
                 if ( typeof(xml) == 'string' ) {
                         t.error({'responseText': xml});
@@ -506 +516 @@
                         updatePending( -1 );
 
                         if ( this.comments_listing == 'moderated' ) {
-                                pid.animate( { 'backgroundColor':'#CCEEBB' }, 400, function(){
-                                        pid.fadeOut();
-                                });
+                                pid.addClass('status-transition' , 400, setTimeout(function () {
+                                        pid.removeClass('status-transition');
+                                }, 400) );
                                 return;
                         }
                 }
 
-                $(c).hide()
+                $(c).hide();
                 $('#replyrow').after(c);
                 id = $(id);
                 t.addEvents(id);
-                bg = id.hasClass('unapproved') ? '#FFFFE0' : id.closest('.widefat, .postbox').css('backgroundColor');
+                bgClass = id.hasClass('unapproved') ? 'unapproved' : 'approved';
 
-                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.removeClass(bgClass);
+                id.addClass('status-transition', 300, setTimeout( function () {
+                        id.removeClass('status-transition');
+                        id.addClass(bgClass);
+                }, 300), function () {
+                        if (pid && pid.length) {
+                                pid.removeClass('unapproved');
+                                pid.addClass('status-transition', 300, setTimeout(function () {
+                                        pid.removeClass('status-transition');
+                        }, 300),
+                        pid.removeClass('unapproved').addClass('approved')
+                        .find('div.comment_status').html('1'));
+                        }
+                });
         },
 
         error : function(r) {

wp-admin/css/wp-admin.css

@@ -1098 +1098 @@
         display: inline;
 }
 
+.status-transition {
+    background-color: #cceebb;
+}
+
 td.action-links,
 th.action-links {
         text-align: right;