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

Unified Diff: chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js

Issue 1457823002: Disable the ChromeVox menu button until it's complete. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 1 month 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: chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js
diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js
index 286fc92b9ab41c2ad4080f2cc7ababa2cf715bcb..365c7c71b4d2f9e9a306f12d05e8aef7c05ae860 100644
--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js
+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/panel.js
@@ -54,10 +54,13 @@ Panel.init = function() {
Panel.exec(/** @type {PanelCommand} */(command));
}, false);
- $('menu').addEventListener('click', Panel.onMenu, false);
$('options').addEventListener('click', Panel.onOptions, false);
$('close').addEventListener('click', Panel.onClose, false);
+ // The ChromeVox menu isn't fully implemented yet, disable it.
+ $('menu').disabled = true;
+ $('triangle').style.display = 'none';
+
Msgs.addTranslatedMessagesToDom(document);
};
@@ -121,14 +124,6 @@ Panel.exec = function(command) {
};
/**
- * Open the ChromeVox Menu.
- */
-Panel.onMenu = function() {
- window.location = '#fullscreen';
- // TODO(dmazzoni): implement the menu UI here.
-};
-
-/**
* Open the ChromeVox Options.
*/
Panel.onOptions = function() {

Powered by Google App Engine
This is Rietveld 408576698