Changeset 147088 in webkit

Timestamp:

Mar 28, 2013 2:33:57 AM (11 years ago)

Author:

commit-queue@webkit.org

Message:

Web Inspector: Fixed property text indentation in Styles sidebar.
​https://bugs.webkit.org/show_bug.cgi?id=113409

Patch by Vladislav Kaznacheev <​kaznacheev@chromium.org> on 2013-03-28
Reviewed by Alexander Pavlov.

When the sidebar is too narrow and the property text wraps around it breaks the indentation.
This was introduced in r143207 which spuriously modified DOM elements order.

• inspector/front-end/StylesSidebarPane.js:

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/front-end/StylesSidebarPane.js (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-03-28  Vladislav Kaznacheev  <kaznacheev@chromium.org>
+
+        Web Inspector: Fixed property text indentation in Styles sidebar.
+        https://bugs.webkit.org/show_bug.cgi?id=113409
+
+        Reviewed by Alexander Pavlov.
+
+        When the sidebar is too narrow and the property text wraps around it breaks the indentation.
+        This was introduced in r143207 which spuriously modified DOM elements order.
+
+        * inspector/front-end/StylesSidebarPane.js:
+
 2013-03-28  Arpita Bahuguna  <a.bah@samsung.com>
 

trunk/Source/WebCore/inspector/front-end/StylesSidebarPane.js

@@ -2146 +2146 @@
             enabledCheckboxElement.checked = !this.disabled;
             enabledCheckboxElement.addEventListener("click", this.toggleEnabled.bind(this), false);
-            this.listItemElement.appendChild(enabledCheckboxElement);
+            this.listItemElement.insertBefore(enabledCheckboxElement, this.listItemElement.firstChild);
         }
     },