Changeset 106236 in webkit

Timestamp:

Jan 30, 2012 5:02:57 AM (12 years ago)

Author:

vsevik@chromium.org

Message:

Web Inspector: Change Open Resource and Go To function shortcuts in scripts panel.
​https://bugs.webkit.org/show_bug.cgi?id=77321

Reviewed by Pavel Feldman.

• inspector/front-end/FilteredItemSelectionDialog.js:

(WebInspector.JavaScriptOutlineDialog.createShortcut):
(WebInspector.OpenResourceDialog.createShortcut):

• inspector/front-end/ScriptsPanel.js:

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/front-end/FilteredItemSelectionDialog.js (modified) (2 diffs)
• inspector/front-end/ScriptsPanel.js (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-01-30  Vsevolod Vlasov  <vsevik@chromium.org>
+
+        Web Inspector: Change Open Resource and Go To function shortcuts in scripts panel.
+        https://bugs.webkit.org/show_bug.cgi?id=77321
+
+        Reviewed by Pavel Feldman.
+
+        * inspector/front-end/FilteredItemSelectionDialog.js:
+        (WebInspector.JavaScriptOutlineDialog.createShortcut):
+        (WebInspector.OpenResourceDialog.createShortcut):
+        * inspector/front-end/ScriptsPanel.js:
+
 2012-01-30  Yury Semikhatsky  <yurys@chromium.org>
 

trunk/Source/WebCore/inspector/front-end/FilteredItemSelectionDialog.js

@@ -516 +516 @@
 WebInspector.JavaScriptOutlineDialog.createShortcut = function()
 {
-    return WebInspector.KeyboardShortcut.makeDescriptor("o", WebInspector.KeyboardShortcut.Modifiers.CtrlOrMeta);
+    return WebInspector.KeyboardShortcut.makeDescriptor("o", WebInspector.KeyboardShortcut.Modifiers.CtrlOrMeta | WebInspector.KeyboardShortcut.Modifiers.Shift);
 }
 
@@ -637 +637 @@
 WebInspector.OpenResourceDialog.createShortcut = function()
 {
-    return WebInspector.KeyboardShortcut.makeDescriptor("t", WebInspector.KeyboardShortcut.Modifiers.CtrlOrMeta | WebInspector.KeyboardShortcut.Modifiers.Shift);
+    return WebInspector.KeyboardShortcut.makeDescriptor("o", WebInspector.KeyboardShortcut.Modifiers.CtrlOrMeta);
 }
 

trunk/Source/WebCore/inspector/front-end/ScriptsPanel.js

@@ -132 +132 @@
     this.registerShortcut(evaluateInConsoleShortcut.key, this._evaluateSelectionInConsole.bind(this));
 
+    var openResourceShortcut = WebInspector.OpenResourceDialog.createShortcut();
+    helpSection.addKey(openResourceShortcut.name, WebInspector.UIString("Open Script"));
+
     var scriptOutlineShortcut = WebInspector.JavaScriptOutlineDialog.createShortcut();
     helpSection.addKey(scriptOutlineShortcut.name, WebInspector.UIString("Go to Function"));
-
-    var openResourceShortcut = WebInspector.OpenResourceDialog.createShortcut();
-    helpSection.addKey(openResourceShortcut.name, WebInspector.UIString("Open Script"));
 
     var panelEnablerHeading = WebInspector.UIString("You need to enable debugging before you can use the Scripts panel.");