Changeset 203994 in webkit

Timestamp:

Aug 1, 2016 4:04:17 PM (8 years ago)

Author:

Nikita Vasilyev

Message:

Web Inspector: Don't set width:calc(100% - 0px); left:0px on console messages
​https://bugs.webkit.org/show_bug.cgi?id=160428
<rdar://problem/27646033>

Reviewed by Timothy Hatcher.

Console messages inside console.group() are indented. Console messages outside of
console.group() aren't and there is no reason to set inline styles that do nothing
on each one of them.

• UserInterface/Views/LogContentView.js:

(WebInspector.LogContentView.prototype.didAppendConsoleMessageView):

Location:

trunk/Source/WebInspectorUI

Files:

• ChangeLog (modified) (1 diff)
• UserInterface/Views/LogContentView.js (modified) (1 diff)

trunk/Source/WebInspectorUI/ChangeLog

@@ +1 @@
+2016-08-01  Nikita Vasilyev  <nvasilyev@apple.com>
+
+        Web Inspector: Don't set `width:calc(100% - 0px); left:0px` on console messages
+        https://bugs.webkit.org/show_bug.cgi?id=160428
+        <rdar://problem/27646033>
+
+        Reviewed by Timothy Hatcher.
+
+        Console messages inside console.group() are indented. Console messages outside of
+        console.group() aren't and there is no reason to set inline styles that do nothing
+        on each one of them.
+
+        * UserInterface/Views/LogContentView.js:
+        (WebInspector.LogContentView.prototype.didAppendConsoleMessageView):
+
 2016-08-01  Matt Baker  <mattbaker@apple.com>
 

trunk/Source/WebInspectorUI/UserInterface/Views/LogContentView.js

@@ -136 +136 @@
         // Nest the message.
         var type = messageView instanceof WebInspector.ConsoleCommandView ? null : messageView.message.type;
-        if (type !== WebInspector.ConsoleMessage.MessageType.EndGroup) {
+        if (this._nestingLevel && type !== WebInspector.ConsoleMessage.MessageType.EndGroup) {
             var x = 16 * this._nestingLevel;
             var messageElement = messageView.element;