Changeset 33982

Timestamp:

09/10/2015 01:02:05 AM (9 years ago)

Author:

wonderboymusic

Message:

After [33821], when dynamically updating the document title text of the Comments List Table page, operate only on the fragment that contains the comment count. This prevents us from including other numbers that may be present in the other title parts (site title, etc).

See #33414.

File:

• trunk/src/wp-admin/js/edit-comments.js (modified) (2 diffs)

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

@@ -139 +139 @@
 
     updateHtmlTitle = function ( diff ) {
-        var newTitle, regExMatch, titleCount;
+        var newTitle, regExMatch, titleCount, commentFrag;
 
         titleRegEx = titleRegEx || new RegExp( 'Comments (\\([0-9' + thousandsSeparator + ']+\\))?' );
@@ -146 +146 @@
         newTitle = adminTitle;
 
-        titleDiv.html( document.title );
-        titleCount = getCount( titleDiv ) + diff;
+        commentFrag = titleRegEx.exec( document.title );
+        if ( commentFrag ) {
+            commentFrag = commentFrag[0];
+            titleDiv.html( commentFrag );
+            titleCount = getCount( titleDiv ) + diff;
+        } else {
+            titleDiv.html( 0 );
+            titleCount = diff;
+        }
+
         if ( titleCount >= 1 ) {
             updateCount( titleDiv, titleCount );