Changeset 33821

Timestamp:

08/31/2015 05:57:32 PM (11 years ago)

Author:

wonderboymusic

Message:

Comments: dynamically update the document title text of the Comments List Table page when dynamically updating the number of comments awaiting moderation.

Fixes #33414.

Location:

trunk/src/wp-admin

Files:

• edit-comments.php (modified) (1 diff)
• js/edit-comments.js (modified) (4 diffs)

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

@@ -105 +105 @@
 enqueue_comment_hotkeys_js();
 
-if ( $post_id )
-        $title = sprintf( __( 'Comments on “%s”' ), wp_html_excerpt( _draft_or_post_title( $post_id ), 50, '…' ) );
-else
-        $title = __('Comments');
+if ( $post_id ) {
+        $comments_count = wp_count_comments( $post_id );
+        $draft_or_post_title = wp_html_excerpt( _draft_or_post_title( $post_id ), 50, '…' );
+        if ( $comments_count->moderated > 0 ) {
+                $title = sprintf(
+                        __( 'Comments (%s) on “%s”' ),
+                        number_format_i18n( $comments_count->moderated ),
+                        $draft_or_post_title
+                );
+        } else {
+                $title = sprintf( __( 'Comments on “%s”' ), $draft_or_post_title );
+        }
+} else {
+        $comments_count = wp_count_comments();
+        if ( $comments_count->moderated > 0 ) {
+                $title = sprintf(
+                        __( 'Comments (%s)' ),
+                        number_format_i18n( $comments_count->moderated )
+                );
+        } else {
+                $title = __( 'Comments' );
+        }
+}
 
 add_screen_option( 'per_page' );

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

@@ -3 +3 @@
 
 (function($) {
-var getCount, updateCount, updateCountText, updatePending, updateApproved;
+var getCount, updateCount, updateCountText, updatePending, updateApproved,
+        updateHtmlTitle, adminTitle = document.title;
 
 setCommentsList = function() {
         var totalInput, perPageInput, pageInput, dimAfter, delBefore, updateTotalCount, delAfter, refillTheExtraList, diff,
-                lastConfidentTime = 0;
+                lastConfidentTime = 0, titleDiv, titleRegEx,
+                isDashboard = $('#dashboard_right_now').length;
 
         totalInput = $('input[name="_total"]', '#comments-form');
@@ -136 +138 @@
         };
 
+        updateHtmlTitle = function ( diff ) {
+                var newTitle, regExMatch, titleCount;
+
+                titleRegEx = titleRegEx || new RegExp( 'Comments (\\([0-9' + thousandsSeparator + ']+\\))?' );
+                // count funcs operate on a $'d element
+                titleDiv = titleDiv || $( '<div />' );
+                newTitle = adminTitle;
+
+                titleDiv.html( document.title );
+                titleCount = getCount( titleDiv ) + diff;
+                if ( titleCount >= 1 ) {
+                        updateCount( titleDiv, titleCount );
+                        regExMatch = titleRegEx.exec( document.title );
+                        if ( regExMatch ) {
+                                newTitle = document.title.replace( regExMatch[0], 'Comments (' + titleDiv.text() + ') ' );
+                        }
+                } else {
+                        regExMatch = titleRegEx.exec( newTitle );
+                        if ( regExMatch ) {
+                                newTitle = newTitle.replace( regExMatch[0], 'Comments' );
+                        }
+                }
+                document.title = newTitle;
+        };
+
         updatePending = function( diff, commentPostId ) {
                 var postSelector = '.post-com-count-' + commentPostId,
@@ -143 +170 @@
                         pending,
                         noPending;
+
+                if ( ! isDashboard ) {
+                        updateHtmlTitle( diff );
+                }
 
                 $( 'span.pending-count' ).each(function() {
@@ -391 +422 @@
                 }
 
-                if ( ! $('#dashboard_right_now').length ) {
+                if ( ! isDashboard ) {
                         total = totalInput.val() ? parseInt( totalInput.val(), 10 ) : 0;
                         if ( $(settings.target).parent().is('span.undo') )