Merge pull request #1307 from adobe/jason-sanjose/shell-dev-tools

Looks good
adobe · Jul 27, 2012 · a2e9807 · a2e9807
2 parents 0161b35 + 22b4d6a
commit a2e9807
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 10 deletions.
diff --git a/src/debug/DebugCommandHandlers.js b/src/debug/DebugCommandHandlers.js
@@ -131,14 +131,15 @@ define(function (require, exports, module) {
         window.open(window.location.href);
     }
 
+    /* Register all the command handlers */
 
-    // Register all the command handlers
-    CommandManager.register(Strings.CMD_SHOW_DEV_TOOLS, Commands.DEBUG_SHOW_DEVELOPER_TOOLS, handleShowDeveloperTools);
-    CommandManager.register(Strings.CMD_RUN_UNIT_TESTS, Commands.DEBUG_RUN_UNIT_TESTS,      _handleRunUnitTests);
-    CommandManager.register(Strings.CMD_SHOW_PERF_DATA, Commands.DEBUG_SHOW_PERF_DATA,      _handleShowPerfData);
-    CommandManager.register(Strings.CMD_NEW_BRACKETS_WINDOW,
-                                                        Commands.DEBUG_NEW_BRACKETS_WINDOW, _handleNewBracketsWindow);
+    // Show Developer Tools (optionally enabled)
+    CommandManager.register(Strings.CMD_SHOW_DEV_TOOLS,      Commands.DEBUG_SHOW_DEVELOPER_TOOLS,   handleShowDeveloperTools)
+        .setEnabled(!!brackets.app.showDeveloperTools);
+    CommandManager.register(Strings.CMD_NEW_BRACKETS_WINDOW, Commands.DEBUG_NEW_BRACKETS_WINDOW,    _handleNewBracketsWindow);
+    CommandManager.register(Strings.CMD_RUN_UNIT_TESTS,      Commands.DEBUG_RUN_UNIT_TESTS,         _handleRunUnitTests);
+    CommandManager.register(Strings.CMD_SHOW_PERF_DATA,      Commands.DEBUG_SHOW_PERF_DATA,         _handleShowPerfData);
 
-    CommandManager.register(Strings.CMD_USE_TAB_CHARS,  Commands.TOGGLE_USE_TAB_CHARS,      _handleUseTabChars)
+    CommandManager.register(Strings.CMD_USE_TAB_CHARS,       Commands.TOGGLE_USE_TAB_CHARS,         _handleUseTabChars)
         .setChecked(Editor.getUseTabChar());
 });
diff --git a/test/SpecRunner.js b/test/SpecRunner.js
@@ -91,9 +91,14 @@ define(function (require, exports, module) {
     }
 
     function _documentReadyHandler() {
-        $("#show-dev-tools").click(function () {
-            brackets.app.showDeveloperTools();
-        });
+        if (brackets.app.showDeveloperTools) {
+            $("#show-dev-tools").click(function () {
+                brackets.app.showDeveloperTools();
+            });
+        } else {
+            $("#show-dev-tools").remove();
+        }
+
         $("#reload").click(function () {
             window.location.reload(true);
         });