Title: [202891] trunk/Source/WebInspectorUI
Revision
202891
Author
timo...@apple.com
Date
2016-07-06 20:25:51 -0700 (Wed, 06 Jul 2016)

Log Message

Web Inspector: REGRESSION: Hitting ⌘T in Web Inspector no longer opens in new tab in Safari
https://bugs.webkit.org/show_bug.cgi?id=159487
rdar://problem/27188047

Change Command-T to Command-Option-T to open a new tab.

I also changed Command-1, etc. to Command-Option-1, etc. to switch tabs. The number commands
conflicted similarly with Safari and didn't work for me as a Web Inspector shortcut at all.
We can't use Command-Shift because of Command-Shift-3 and Command-Shift-4 for screenshots.

Reviewed by Joseph Pecoraro.

* UserInterface/Base/Main.js:
(WebInspector.contentLoaded):

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (202890 => 202891)


--- trunk/Source/WebInspectorUI/ChangeLog	2016-07-07 03:12:37 UTC (rev 202890)
+++ trunk/Source/WebInspectorUI/ChangeLog	2016-07-07 03:25:51 UTC (rev 202891)
@@ -1,3 +1,20 @@
+2016-07-06  Timothy Hatcher  <timo...@apple.com>
+
+        Web Inspector: REGRESSION: Hitting ⌘T in Web Inspector no longer opens in new tab in Safari
+        https://bugs.webkit.org/show_bug.cgi?id=159487
+        rdar://problem/27188047
+
+        Change Command-T to Command-Option-T to open a new tab.
+
+        I also changed Command-1, etc. to Command-Option-1, etc. to switch tabs. The number commands
+        conflicted similarly with Safari and didn't work for me as a Web Inspector shortcut at all.
+        We can't use Command-Shift because of Command-Shift-3 and Command-Shift-4 for screenshots.
+
+        Reviewed by Joseph Pecoraro.
+
+        * UserInterface/Base/Main.js:
+        (WebInspector.contentLoaded):
+
 2016-07-06  Joseph Pecoraro  <pecor...@apple.com>
 
         Web Inspector: sometimes reloading a page with main resource selected will show an empty content view

Modified: trunk/Source/WebInspectorUI/UserInterface/Base/Main.js (202890 => 202891)


--- trunk/Source/WebInspectorUI/UserInterface/Base/Main.js	2016-07-07 03:12:37 UTC (rev 202890)
+++ trunk/Source/WebInspectorUI/UserInterface/Base/Main.js	2016-07-07 03:25:51 UTC (rev 202891)
@@ -268,8 +268,8 @@
     this._decreaseZoomKeyboardShortcut = new WebInspector.KeyboardShortcut(WebInspector.KeyboardShortcut.Modifier.CommandOrControl, WebInspector.KeyboardShortcut.Key.Minus, this._decreaseZoom.bind(this));
     this._resetZoomKeyboardShortcut = new WebInspector.KeyboardShortcut(WebInspector.KeyboardShortcut.Modifier.CommandOrControl, "0", this._resetZoom.bind(this));
 
-    this._showTabAtIndexKeyboardShortcuts = [1, 2, 3, 4, 5, 6, 7, 8, 9].map((i) => new WebInspector.KeyboardShortcut(WebInspector.KeyboardShortcut.Modifier.CommandOrControl, `${i}`, this._showTabAtIndex.bind(this, i)));
-    this._openNewTabKeyboardShortcut = new WebInspector.KeyboardShortcut(WebInspector.KeyboardShortcut.Modifier.CommandOrControl, "T", this.showNewTabTab.bind(this));
+    this._showTabAtIndexKeyboardShortcuts = [1, 2, 3, 4, 5, 6, 7, 8, 9].map((i) => new WebInspector.KeyboardShortcut(WebInspector.KeyboardShortcut.Modifier.CommandOrControl | WebInspector.KeyboardShortcut.Modifier.Option, `${i}`, this._showTabAtIndex.bind(this, i)));
+    this._openNewTabKeyboardShortcut = new WebInspector.KeyboardShortcut(WebInspector.KeyboardShortcut.Modifier.CommandOrControl | WebInspector.KeyboardShortcut.Modifier.Option, "T", this.showNewTabTab.bind(this));
 
     this.tabBrowser = new WebInspector.TabBrowser(document.getElementById("tab-browser"), this.tabBar, this.navigationSidebar, this.detailsSidebar);
     this.tabBrowser.addEventListener(WebInspector.TabBrowser.Event.SelectedTabContentViewDidChange, this._tabBrowserSelectedTabContentViewDidChange, this);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to