Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1382)

Unified Diff: chrome/browser/resources/file_manager/js/file_manager.js

Issue 14799005: Fix legacy UI in Files.app. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/file_manager/js/file_manager.js
diff --git a/chrome/browser/resources/file_manager/js/file_manager.js b/chrome/browser/resources/file_manager/js/file_manager.js
index c68904fabde0359603227f872cecda417d014bc5..26af4c25459643db374cb0910a1a8abd8a9c8a17 100644
--- a/chrome/browser/resources/file_manager/js/file_manager.js
+++ b/chrome/browser/resources/file_manager/js/file_manager.js
@@ -1060,9 +1060,11 @@ DialogType.isModal = function(type) {
FileManager.prototype.initSidebar_ = function() {
this.directoryTree_ = this.dialogDom_.querySelector('#directory-tree');
DirectoryTree.decorate(this.directoryTree_, this.directoryModel_);
- this.directoryTree_.addEventListener('content-updated', function() {
- this.updateMiddleBarVisibility_(true);
- }.bind(this));
+ if (util.platform.newUI()) {
+ this.directoryTree_.addEventListener('content-updated', function() {
+ this.updateMiddleBarVisibility_(true);
+ }.bind(this));
+ }
if (util.platform.newUI()) {
this.volumeList_ = this.dialogDom_.querySelector('#volume-list');
VolumeList.decorate(this.volumeList_, this.directoryModel_);
@@ -1424,14 +1426,14 @@ DialogType.isModal = function(type) {
this.table_.normalizeColumns();
}
this.table_.redraw();
- this.volumeList_.redraw();
}
if (!util.platform.newUI())
this.breadcrumbs_.truncate();
+ else
+ this.volumeList_.redraw();
this.searchBreadcrumbs_.truncate();
-
this.updateWindowState_();
};
@@ -2539,7 +2541,8 @@ DialogType.isModal = function(type) {
}
this.table_.list.endBatchUpdates();
this.grid_.endBatchUpdates();
- this.updateMiddleBarVisibility_();
+ if (!util.platform.newUI())
+ this.updateMiddleBarVisibility_();
yoshiki 2013/05/13 02:00:48 This is necessary on New UI.
mtomasz 2013/05/13 02:07:38 Fixed. Done.
this.scanCompletedTimer_ = null;
}.bind(this), 50);
};
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698