Web Inspector: Elements: Computed: move the Box Model section to the top
authordrousso@apple.com <drousso@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 14 Aug 2019 18:45:05 +0000 (18:45 +0000)
committerdrousso@apple.com <drousso@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 14 Aug 2019 18:45:05 +0000 (18:45 +0000)
https://bugs.webkit.org/show_bug.cgi?id=200555

Reviewed by Joseph Pecoraro.

For nodes that use a lot of different CSS properties, having the Box Model section all the
way at the bottom isn't as useful for taking a quick glance at the node's box model data.

* UserInterface/Views/ComputedStyleDetailsPanel.js:
(WI.ComputedStyleDetailsPanel.prototype.initialLayout):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@248683 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebInspectorUI/ChangeLog
Source/WebInspectorUI/UserInterface/Views/ComputedStyleDetailsPanel.js

index 0831dd2..4616b03 100644 (file)
@@ -1,5 +1,18 @@
 2019-08-14  Devin Rousso  <drousso@apple.com>
 
+        Web Inspector: Elements: Computed: move the Box Model section to the top
+        https://bugs.webkit.org/show_bug.cgi?id=200555
+
+        Reviewed by Joseph Pecoraro.
+
+        For nodes that use a lot of different CSS properties, having the Box Model section all the
+        way at the bottom isn't as useful for taking a quick glance at the node's box model data.
+
+        * UserInterface/Views/ComputedStyleDetailsPanel.js:
+        (WI.ComputedStyleDetailsPanel.prototype.initialLayout):
+
+2019-08-14  Devin Rousso  <drousso@apple.com>
+
         Web Inspector: Elements: Computed: the collapsed state of the Variables section should be separate from the collapsed state of the Properties section
         https://bugs.webkit.org/show_bug.cgi?id=200725
 
index 8f86296..bde139d 100644 (file)
@@ -82,6 +82,13 @@ WI.ComputedStyleDetailsPanel = class ComputedStyleDetailsPanel extends WI.StyleD
 
     initialLayout()
     {
+        this._boxModelDiagramRow = new WI.BoxModelDetailsSectionRow;
+
+        let boxModelGroup = new WI.DetailsSectionGroup([this._boxModelDiagramRow]);
+        let boxModelSection = new WI.DetailsSection("computed-style-box-model", WI.UIString("Box Model"), [boxModelGroup]);
+
+        this.element.appendChild(boxModelSection.element);
+
         let computedStyleShowAllLabel = document.createElement("label");
         computedStyleShowAllLabel.textContent = WI.UIString("Show All");
 
@@ -125,13 +132,6 @@ WI.ComputedStyleDetailsPanel = class ComputedStyleDetailsPanel extends WI.StyleD
 
         this.element.appendChild(this._propertiesSection.element);
         this.element.appendChild(this._variablesSection.element);
-
-        this._boxModelDiagramRow = new WI.BoxModelDetailsSectionRow;
-
-        let boxModelGroup = new WI.DetailsSectionGroup([this._boxModelDiagramRow]);
-        let boxModelSection = new WI.DetailsSection("computed-style-box-model", WI.UIString("Box Model"), [boxModelGroup]);
-
-        this.element.appendChild(boxModelSection.element);
     }
 
     filterDidChange(filterBar)