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

Unified 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 side-by-side diff with in-line comments
Download patch
Index: ui/file_manager/file_manager/foreground/js/ui/file_manager_ui.js
diff --git a/ui/file_manager/file_manager/foreground/js/ui/file_manager_ui.js b/ui/file_manager/file_manager/foreground/js/ui/file_manager_ui.js
index 83464b92e9603aaa69088d88d03adf01e78ab7e4..164175e746d55cde14a465fb39f44a6bb7a5d32f 100644
--- a/ui/file_manager/file_manager/foreground/js/ui/file_manager_ui.js
+++ b/ui/file_manager/file_manager/foreground/js/ui/file_manager_ui.js
@@ -128,28 +128,11 @@ function FileManagerUI(element, launchParam) {
this.locationLine = null;
/**
- * The toolbar button to cancel selection.
+ * The toolbar which contains controls.
* @type {!HTMLElement}
* @const
*/
- this.cancelSelectionButton =
- queryRequiredElement(this.element, '#cancel-selection-button');
-
- /**
- * The wrapper for the toolbar button to cancel selection.
- * @type {!HTMLElement}
- * @const
- */
- this.cancelSelectionButtonWrapper =
- queryRequiredElement(this.element, '#cancel-selection-button-wrapper');
-
- /**
- * The label showing how many files are selected.
- * @type {!HTMLElement}
- * @const
- */
- this.filesSelectedLabel =
- queryRequiredElement(this.element, '#files-selected-label');
+ this.toolbar = queryRequiredElement(this.element, '.dialog-header');
/**
* The navigation list.
« 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