Add horizontal scrollbar to long request headers as soon as they are

displayed instead of their display being toggled (related to issue 6445)
firebug · Feb 17, 2014 · 5c7ef0b · simonlindholm · Feb 17, 2014 · SebastianZ
1 parent 9bd099e
commit 5c7ef0b
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/extension/content/firebug/lib/dom.js b/extension/content/firebug/lib/dom.js
@@ -543,7 +543,6 @@ Dom.getScrollBarWidth = function(doc)
     doc.body.removeChild(outer);
 
     scrollBarWidth = widthWithoutScrollbar - widthWithScrollbar;
-    FBTrace.sysout("scrollBarWidth "+scrollBarWidth);
     return scrollBarWidth;
 };
 

diff --git a/extension/content/firebug/net/netPanel.js b/extension/content/firebug/net/netPanel.js
@@ -809,7 +809,7 @@ NetPanel.prototype = Obj.extend(ActivablePanel,
         }
     },
 
-    onToggleSourceView: function()
+    onHeadersDisplayed: function()
     {
         this.updateHeadersSourceWidth();
     },

diff --git a/extension/content/firebug/net/netReps.js b/extension/content/firebug/net/netReps.js
@@ -1102,6 +1102,9 @@ Firebug.NetMonitor.NetInfoBody = domplate(Rep, new EventSource(),
             }
         }
 
+        // Notify listeners, so additional actions can be done when the headers are displayed
+        Events.dispatch(Firebug.NetMonitor.NetInfoBody.fbListeners, "onHeadersDisplayed", [this]);
+
         if (Css.hasClass(tab, "netInfoPostTab"))
         {
             if (!netInfoBox.postPresented)
@@ -1724,10 +1727,6 @@ Firebug.NetMonitor.NetInfoHeaders = domplate(Rep, new EventSource(),
 
         target.sourceDisplayed = !target.sourceDisplayed;
 
-        // Notify listeners, so additional actions can be done when source view was toggled
-        Events.dispatch(Firebug.NetMonitor.NetInfoBody.fbListeners, "onToggleSourceView",
-            [target.sourceDisplayed]);
-
         Events.cancelEvent(event);
     },