Web Inspector: Console: timestamps cause objects to be shown on a sep…

…arate line https://bugs.webkit.org/show_bug.cgi?id=255031 <rdar://problem/107659829> Reviewed by Patrick Angle. Move the timestamp to the end of the line and `float` it so that it doesn't participate in the normal flow. * Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js: (WI.ConsoleMessageView.prototype.renderTimestamp): * Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.css: (.console-message .timestamp): Canonical link: https://commits.webkit.org/263263@main
WebKit · Apr 21, 2023 · 2f8d67e · 2f8d67e
1 parent 307a535
commit 2f8d67e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.css b/Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.css
@@ -311,8 +311,9 @@
 }
 
 .console-message .timestamp {
+    float: right;
+    margin-inline-start: 12px;
     color: var(--text-color-tertiary);
-    padding-inline-end: 5px;
 }
 
 .console-session:not(.timestamps-visible) .console-message .timestamp {

diff --git a/Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js b/Source/WebInspectorUI/UserInterface/Views/ConsoleMessageView.js
@@ -187,7 +187,7 @@ WI.ConsoleMessageView = class ConsoleMessageView extends WI.Object
         if (!this._timestampElement) {
             this._timestampElement = document.createElement("span");
             this._timestampElement.classList.add("timestamp");
-            this._messageBodyElement.insertBefore(this._timestampElement, this._messageBodyElement.firstChild);
+            this._element.insertBefore(this._timestampElement, this._element.firstChild);
         }
 
         let date = new Date(this._timestamp * 1000);