Changeset 33692

Timestamp:

08/21/2015 05:25:06 PM (9 years ago)

Author:

wonderboymusic

Message:

Comment List Tables:

• Toggle visibility of pending bubble when a comment is dynamically moderated
• Add a CSS class to the pending bubble to hide it / remove to show it

See #11200.

Location:

trunk/src/wp-admin

Files:

• css/list-tables.css (modified) (1 diff)
• includes/class-wp-list-table.php (modified) (1 diff)
• js/edit-comments.js (modified) (5 diffs)

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

@@ -118 +118 @@
 }
 
+.column-response .post-com-count-no-pending,
+.column-comments .post-com-count-no-pending {
+    display: none;
+}
+
 /* comments */
 

trunk/src/wp-admin/includes/class-wp-list-table.php

@@ -650 +650 @@
             );
         } else {
-            printf( '<span class="post-com-count post-com-count-pending"><span class="comment-count comment-count-no-pending" aria-hidden="true">%s</span><span class="screen-reader-text">%s</span></span>',
+            printf( '<span class="post-com-count post-com-count-pending post-com-count-no-pending"><span class="comment-count comment-count-no-pending" aria-hidden="true">%s</span><span class="screen-reader-text">%s</span></span>',
                 $pending_comments_number,
                 $approved_comments ? __( 'No pending comments' ) : __( 'No comments' )

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

@@ -139 +139 @@
         var postSelector = '.post-com-count-' + commentPostId,
             noClass = 'comment-count-no-pending',
+            noParentClass = 'post-com-count-no-pending',
             pendingClass = 'comment-count-pending',
-            counts = $( 'span.pending-count' ),
             pending,
             noPending;
 
-        counts.each(function() {
+        $( 'span.pending-count' ).each(function() {
             var a = $(this), n = getCount(a) + diff;
             if ( n < 1 )
@@ -166 +166 @@
 
             if ( 0 === n ) {
+                a.parent().addClass( noParentClass );
                 a.removeClass( pendingClass ).addClass( noClass );
+            } else {
+                a.parent().removeClass( noParentClass );
+                a.addClass( pendingClass ).removeClass( noClass );
             }
             updateCount( a, n );
@@ -174 +178 @@
             var a = $(this);
             if ( diff > 0 ) {
+                a.parent().removeClass( noParentClass );
                 a.removeClass( noClass ).addClass( pendingClass );
+            } else {
+                a.parent().addClass( noParentClass );
+                a.addClass( noClass ).removeClass( pendingClass );
             }
             updateCount( a, diff );
@@ -204 +212 @@
             if ( 0 === n ) {
                 a.removeClass( approvedClass ).addClass( noClass );
+            } else {
+                a.addClass( approvedClass ).removeClass( noClass );
             }
             updateCount( a, n );
@@ -212 +222 @@
             if ( diff > 0 ) {
                 a.removeClass( noClass ).addClass( approvedClass );
+            } else {
+                a.addClass( noClass ).removeClass( approvedClass );
             }
             updateCount( a, diff );