<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[245696] trunk/Source/WebInspectorUI</title>
</head>
<body>

<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }
#msg dl a { font-weight: bold}
#msg dl a:link    { color:#fc3; }
#msg dl a:active  { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/245696">245696</a></dd>
<dt>Author</dt> <dd>drousso@apple.com</dd>
<dt>Date</dt> <dd>2019-05-23 11:04:49 -0700 (Thu, 23 May 2019)</dd>
</dl>

<h3>Log Message</h3>
<pre>Web Inspector: clicking a source link should never open the Network tab
https://bugs.webkit.org/show_bug.cgi?id=197951

Reviewed by Timothy Hatcher.

The preview panel in the Network tab is really meant to be just that; a preview.

Opening resources in the preview panel is not a great experience, as the Network tab doesn't
have as much of the information one would expect when viewing a resource (namely the
Resource details sidebar panel).

Navigating within the Network tab also completely wipes out the current "state" of what
you're looking at (e.g. which panel), which could be unexpected.

The only way this could happen previously was if the Resources/Debugger/Sources tabs had
never been active at any point in the past. Selecting one of those tabs and then clicking on
an initiator link in the Network tab would re-select that other tab.

* UserInterface/Base/Main.js:
(WI.dialogWasDismissedWithRepresentedObject):

* UserInterface/Views/ResourceHeadersContentView.js:
(WI.ResourceHeadersContentView.prototype._refreshSummarySection):</pre>

<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkSourceWebInspectorUIChangeLog">trunk/Source/WebInspectorUI/ChangeLog</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceBaseMainjs">trunk/Source/WebInspectorUI/UserInterface/Base/Main.js</a></li>
<li><a href="#trunkSourceWebInspectorUIUserInterfaceViewsResourceHeadersContentViewjs">trunk/Source/WebInspectorUI/UserInterface/Views/ResourceHeadersContentView.js</a></li>
</ul>

</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkSourceWebInspectorUIChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/ChangeLog (245695 => 245696)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/ChangeLog    2019-05-23 18:02:13 UTC (rev 245695)
+++ trunk/Source/WebInspectorUI/ChangeLog       2019-05-23 18:04:49 UTC (rev 245696)
</span><span class="lines">@@ -1,3 +1,29 @@
</span><ins>+2019-05-23  Devin Rousso  <drousso@apple.com>
+
+        Web Inspector: clicking a source link should never open the Network tab
+        https://bugs.webkit.org/show_bug.cgi?id=197951
+
+        Reviewed by Timothy Hatcher.
+
+        The preview panel in the Network tab is really meant to be just that; a preview.
+
+        Opening resources in the preview panel is not a great experience, as the Network tab doesn't
+        have as much of the information one would expect when viewing a resource (namely the
+        Resource details sidebar panel).
+
+        Navigating within the Network tab also completely wipes out the current "state" of what
+        you're looking at (e.g. which panel), which could be unexpected.
+
+        The only way this could happen previously was if the Resources/Debugger/Sources tabs had
+        never been active at any point in the past. Selecting one of those tabs and then clicking on
+        an initiator link in the Network tab would re-select that other tab.
+
+        * UserInterface/Base/Main.js:
+        (WI.dialogWasDismissedWithRepresentedObject):
+
+        * UserInterface/Views/ResourceHeadersContentView.js:
+        (WI.ResourceHeadersContentView.prototype._refreshSummarySection):
+
</ins><span class="cx"> 2019-05-22  Nikita Vasilyev  <nvasilyev@apple.com>
</span><span class="cx"> 
</span><span class="cx">         Web Inspector: Provide UIString descriptions to improve localizations
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceBaseMainjs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Base/Main.js (245695 => 245696)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Base/Main.js   2019-05-23 18:02:13 UTC (rev 245695)
+++ trunk/Source/WebInspectorUI/UserInterface/Base/Main.js      2019-05-23 18:04:49 UTC (rev 245696)
</span><span class="lines">@@ -3235,7 +3235,10 @@
</span><span class="cx">     if (!representedObject)
</span><span class="cx">         return;
</span><span class="cx"> 
</span><del>-    WI.showRepresentedObject(representedObject, dialog.cookie);
</del><ins>+    WI.showRepresentedObject(representedObject, dialog.cookie, {
+        ignoreSearchTab: true,
+        ignoreNetworkTab: true,
+    });
</ins><span class="cx"> };
</span><span class="cx"> 
</span><span class="cx"> // Popover delegate
</span></span></pre></div>
<a id="trunkSourceWebInspectorUIUserInterfaceViewsResourceHeadersContentViewjs"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ResourceHeadersContentView.js (245695 => 245696)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebInspectorUI/UserInterface/Views/ResourceHeadersContentView.js    2019-05-23 18:02:13 UTC (rev 245695)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ResourceHeadersContentView.js       2019-05-23 18:04:49 UTC (rev 245696)
</span><span class="lines">@@ -270,6 +270,7 @@
</span><span class="cx">             const options = {
</span><span class="cx">                 dontFloat: true,
</span><span class="cx">                 ignoreSearchTab: true,
</span><ins>+                ignoreNetworkTab: true,
</ins><span class="cx">             };
</span><span class="cx">             let link = WI.createSourceCodeLocationLink(initiatorLocation, options);
</span><span class="cx">             fragment.appendChild(link);
</span><span class="lines">@@ -283,7 +284,7 @@
</span><span class="cx">                 this._popoverCallStackIconElement.addEventListener("click", (event) => {
</span><span class="cx">                     if (!this._popover) {
</span><span class="cx">                         this._popover = new WI.Popover(this);
</span><del>-                        this._popover.windowResizeHandler = () => { this._presentPopoverBelowCallStackElement() };
</del><ins>+                        this._popover.windowResizeHandler = () => { this._presentPopoverBelowCallStackElement(); };
</ins><span class="cx">                     }
</span><span class="cx"> 
</span><span class="cx">                     const selectable = false;
</span></span></pre>
</div>
</div>

</body>
</html>