Chromium Code Reviews| Index: chrome/browser/resources/file_manager/js/file_manager.js |
| =================================================================== |
| --- chrome/browser/resources/file_manager/js/file_manager.js (revision 137095) |
| +++ chrome/browser/resources/file_manager/js/file_manager.js (working copy) |
| @@ -4337,15 +4337,31 @@ |
| }; |
| FileManager.prototype.onSearchBoxUpdate_ = function(event) { |
| - var searchString = this.dialogDom_.querySelector('#search-box').value; |
| + var searchString = this.document_.getElementById('search-box').value; |
|
SeRya
2012/05/15 14:08:03
var searchBox = this.document_.getElementById('sea
Oleg Eterevsky
2012/05/15 14:16:51
Why? I don't need it.
|
| if (searchString) { |
| this.directoryModel_.addFilter( |
| 'searchbox', |
| function(e) { |
| return e.name.substr(0, searchString.length) == searchString; |
| }); |
| + |
| + this.reportEmptySearchResults_ = function() { |
| + if (this.directoryModel_.getFileList().length === 0) { |
| + var text = strf('SEARCH_NO_MATCHING_FILES', searchString); |
| + this.document_.getElementById('no-search-results').innerHTML = text; |
| + this.document_.getElementById('no-search-results').removeAttribute( |
|
SeRya
2012/05/15 14:08:03
.hidden = false;
Oleg Eterevsky
2012/05/15 14:16:51
Done.
|
| + 'hidden'); |
| + } |
| + }.bind(this); |
| + |
| + this.directoryModel_.addEventListener( |
| + 'rescan-completed', this.reportEmptySearchResults_); |
|
SeRya
2012/05/15 14:08:03
The 'splice' event of fileList is better because i
Oleg Eterevsky
2012/05/15 14:16:51
I don't believe there is any difference.
|
| } else { |
| this.directoryModel_.removeFilter('searchbox'); |
| + this.document_.getElementById('no-search-results').setAttribute('hidden', |
|
SeRya
2012/05/15 14:08:03
There is a setter for 'hidden'. Use ".hidden = tru
Oleg Eterevsky
2012/05/15 14:16:51
Done.
|
| + 'true'); |
| + this.directoryModel_.removeEventListener( |
| + 'rescan-completed', this.reportEmptySearchResults_); |
| } |
| }; |