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

Side by Side Diff: ui/file_manager/file_manager/foreground/js/ui/file_manager_ui.js

Issue 951443002: Files.app: Make delete button on the toolbar standard button instead of CommandButton. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add @const for const members of ToolbarController. Created 5 years, 10 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * The root of the file manager's view managing the DOM of Files.app. 6 * The root of the file manager's view managing the DOM of Files.app.
7 * 7 *
8 * @param {!HTMLElement} element Top level element of Files.app. 8 * @param {!HTMLElement} element Top level element of Files.app.
9 * @param {!LaunchParam} launchParam Launch param. 9 * @param {!LaunchParam} launchParam Launch param.
10 * @constructor 10 * @constructor
(...skipping 110 matching lines...) Expand 10 before | Expand all | Expand 10 after
121 this.textContextMenu = util.queryDecoratedElement( 121 this.textContextMenu = util.queryDecoratedElement(
122 '#text-context-menu', cr.ui.Menu); 122 '#text-context-menu', cr.ui.Menu);
123 123
124 /** 124 /**
125 * Location line. 125 * Location line.
126 * @type {LocationLine} 126 * @type {LocationLine}
127 */ 127 */
128 this.locationLine = null; 128 this.locationLine = null;
129 129
130 /** 130 /**
131 * The toolbar button to cancel selection. 131 * The toolbar which contains controls.
132 * @type {!HTMLElement} 132 * @type {!HTMLElement}
133 * @const 133 * @const
134 */ 134 */
135 this.cancelSelectionButton = 135 this.toolbar = queryRequiredElement(this.element, '.dialog-header');
136 queryRequiredElement(this.element, '#cancel-selection-button');
137
138 /**
139 * The wrapper for the toolbar button to cancel selection.
140 * @type {!HTMLElement}
141 * @const
142 */
143 this.cancelSelectionButtonWrapper =
144 queryRequiredElement(this.element, '#cancel-selection-button-wrapper');
145
146 /**
147 * The label showing how many files are selected.
148 * @type {!HTMLElement}
149 * @const
150 */
151 this.filesSelectedLabel =
152 queryRequiredElement(this.element, '#files-selected-label');
153 136
154 /** 137 /**
155 * The navigation list. 138 * The navigation list.
156 * @type {!HTMLElement} 139 * @type {!HTMLElement}
157 * @const 140 * @const
158 */ 141 */
159 this.dialogNavigationList = 142 this.dialogNavigationList =
160 queryRequiredElement(this.element, '.dialog-navigation-list'); 143 queryRequiredElement(this.element, '.dialog-navigation-list');
161 144
162 /** 145 /**
(...skipping 285 matching lines...) Expand 10 before | Expand all | Expand 10 after
448 }, 431 },
449 432
450 handleSplitterDragEnd: function(e) { 433 handleSplitterDragEnd: function(e) {
451 Splitter.prototype.handleSplitterDragEnd.apply(this, arguments); 434 Splitter.prototype.handleSplitterDragEnd.apply(this, arguments);
452 this.ownerDocument.documentElement.classList.remove('col-resize'); 435 this.ownerDocument.documentElement.classList.remove('col-resize');
453 } 436 }
454 }; 437 };
455 438
456 customSplitter.decorate(splitterElement); 439 customSplitter.decorate(splitterElement);
457 }; 440 };
OLDNEW
« no previous file with comments | « ui/file_manager/file_manager/foreground/js/toolbar_controller.js ('k') | ui/file_manager/file_manager/main.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698