Changeset 37967

Timestamp:

07/05/2016 02:37:06 PM (4 years ago)

Author:

ocean90

Message:

Themes: Avoid announcing the theme search results too many times.

Props afercia.
Fixes #36848.

File:

• trunk/src/wp-admin/js/theme.js (modified) (3 diffs)

trunk/src/wp-admin/js/theme.js

@@ -1104 +1104 @@
         this.count.text( this.liveThemeCount );
 
-        this.announceSearchResults( this.liveThemeCount );
+        /*
+         * In the theme installer the themes count is already announced
+         * because `announceSearchResults` is called on `query:success`.
+         */
+        if ( ! themes.isInstall ) {
+            this.announceSearchResults( this.liveThemeCount );
+        }
     },
 
@@ -1460 +1466 @@
     },
 
+    terms: '',
+
     // Handles Ajax request for searching through themes in public repo
     search: function( event ) {
@@ -1480 +1488 @@
     doSearch: _.debounce( function( value ) {
         var request = {};
+
+        // Don't do anything if the search terms haven't changed.
+        if ( this.terms === value ) {
+            return;
+        }
+
+        // Updates terms with the value passed.
+        this.terms = value;
 
         request.search = value;