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

Unified Diff: ui/file_manager/file_manager/foreground/js/file_manager.js

Issue 1223693002: Add tooltips to the header buttons in Files app. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix tests. Created 5 years, 5 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/file_manager.js
diff --git a/ui/file_manager/file_manager/foreground/js/file_manager.js b/ui/file_manager/file_manager/foreground/js/file_manager.js
index 7c80240c2ad855c8553b96ac52e4a0353630f122..acf7b8d3eeb4aa115c9732ece0e37af4829ec86b 100644
--- a/ui/file_manager/file_manager/foreground/js/file_manager.js
+++ b/ui/file_manager/file_manager/foreground/js/file_manager.js
@@ -200,6 +200,13 @@ function FileManager() {
this.toolbarController_ = null;
/**
+ * Tooltip controller.
+ * @type {TooltipController}
+ * @private
+ */
+ this.tooltipController_ = null;
+
+ /**
* Empty folder controller.
* @private {EmptyFolderController}
*/
@@ -425,6 +432,7 @@ FileManager.prototype = /** @struct */ {
assert(this.selectionHandler_);
assert(this.launchParams_);
assert(this.volumeManager_);
+ assert(this.dialogDom_);
this.scanController_ = new ScanController(
this.directoryModel_,
@@ -449,6 +457,10 @@ FileManager.prototype = /** @struct */ {
assert(this.ui_.locationLine),
this.selectionHandler_,
this.directoryModel_);
+ this.tooltipController_ = new TooltipController(
+ queryRequiredElement(this.dialogDom_, '#tooltip'),
+ Array.prototype.slice.call(
+ this.dialogDom_.querySelectorAll('[has-tooltip]')));
this.emptyFolderController_ = new EmptyFolderController(
this.ui_.emptyFolder,
this.directoryModel_);

Powered by Google App Engine
This is Rietveld 408576698