Attachment 360776 Details for Bug 194109 – [PATCH] Proposed Fix

[patch] [PATCH] Proposed Fix

timeline-duration-1.patch (text/plain), 2.11 KB, created by Joseph Pecoraro on 2019-01-31 14:12:14 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Joseph Pecoraro

Created: 2019-01-31 14:12:14 PST

Size: 2.11 KB

patch

obsolete

>diff --git a/Source/WebInspectorUI/ChangeLog b/Source/WebInspectorUI/ChangeLog
>index e1d4d35137f..aa65fefff79 100644
>--- a/Source/WebInspectorUI/ChangeLog
>+++ b/Source/WebInspectorUI/ChangeLog
>@@ -1,3 +1,15 @@
>+2019-01-31  Joseph Pecoraro  <pecoraro@apple.com>
>+
>+        Web Inspector: Timeline time range selection should show duration alongside start and end
>+        https://bugs.webkit.org/show_bug.cgi?id=194109
>+        <rdar://problem/47714279>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * UserInterface/Views/TimelineRecordingContentView.js:
>+        (WI.TimelineRecordingContentView.prototype._updateTimeRangePathComponents):
>+        Include the duration when not obvious.
>+
> 2019-01-31  Joseph Pecoraro  <pecoraro@apple.com>
> 
>         Web Inspector: Timeline time range selection sometimes shows 0.000, should be just 0
>diff --git a/Source/WebInspectorUI/UserInterface/Views/TimelineRecordingContentView.js b/Source/WebInspectorUI/UserInterface/Views/TimelineRecordingContentView.js
>index f54c76775bb..029d7e48543 100644
>--- a/Source/WebInspectorUI/UserInterface/Views/TimelineRecordingContentView.js
>+++ b/Source/WebInspectorUI/UserInterface/Views/TimelineRecordingContentView.js
>@@ -731,7 +731,13 @@ WI.TimelineRecordingContentView = class TimelineRecordingContentView extends WI.
>         if (this._timelineOverview.viewMode === WI.TimelineOverview.ViewMode.Timelines) {
>             let selectionStart = Number.secondsToString(startValue, true);
>             let selectionEnd = Number.secondsToString(endValue, true);
>-            displayName = WI.UIString("%s \u2013 %s").format(selectionStart, selectionEnd);
>+            const epsilon = 0.0001;
>+            if (startValue < epsilon)
>+                displayName = WI.UIString("%s \u2013 %s").format(selectionStart, selectionEnd);
>+            else {
>+                let duration = Number.secondsToString(endValue - startValue, true);
>+                displayName = WI.UIString("%s \u2013 %s (%s)").format(selectionStart, selectionEnd, duration);
>+            }
>         } else {
>             startValue += 1; // Convert index to frame number.
>             if (startValue === endValue)

Actions: View | Formatted Diff | Diff

Attachments on bug 194109: 360776 | 360777 | 360778