sergiotarxz
/
Wordpress
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

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

Props afercia.
Fixes #36848.

git-svn-id: https://develop.svn.wordpress.org/trunk@37967 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Dominik Schilling 2016-07-05 14:37:06 +00:00
parent 727b070c60
commit 13896fc5de
1 changed files with 17 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/wp-admin/js/theme.js
View File

@ -1103,7 +1103,13 @@ themes.view.Themes = wp.Backbone.View.extend({
this.liveThemeCount = this.collection.count ? this.collection.count : this.collection.length;
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 );
}
},
// Iterates through each instance of the collection
@ -1459,6 +1465,8 @@ themes.view.InstallerSearch = themes.view.Search.extend({
'keyup': 'search'
},
terms: '',
// Handles Ajax request for searching through themes in public repo
search: function( event ) {
@ -1480,6 +1488,14 @@ themes.view.InstallerSearch = themes.view.Search.extend({
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;
// Intercept an [author] search.