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);