Attachment 258910 Details for Bug 147650 – [Patch] Proposed Fix

[patch] [Patch] Proposed Fix

bug-147650-20150813120601.patch (text/plain), 1.66 KB, created by Matt Baker on 2015-08-13 12:06:19 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Matt Baker

Created: 2015-08-13 12:06:19 PDT

Size: 1.66 KB

patch

obsolete

>Subversion Revision: 188387
>diff --git a/Source/WebInspectorUI/ChangeLog b/Source/WebInspectorUI/ChangeLog
>index b0452215cf50983bf65ad6049c774a232c32d303..06494585070c431c4e35976d6dcbc6a07fbd41da 100644
>--- a/Source/WebInspectorUI/ChangeLog
>+++ b/Source/WebInspectorUI/ChangeLog
>@@ -1,3 +1,16 @@
>+2015-08-13  Matt Baker  <mattbaker@apple.com>
>+
>+        Web Inspector: Clearing frames timeline doesn't remove current time marker
>+        https://bugs.webkit.org/show_bug.cgi?id=147650
>+
>+        Reviewed by Timothy Hatcher.
>+
>+        The rendering frames timeline offsets all markers by 1px to align them on frame
>+        boundaries, which causes the current time marker to be visible even with left: 0px.
>+        We can exclude the current time marker without it being noticable during recording.
>+
>+        * UserInterface/Views/TimelineOverview.css:
>+
> 2015-08-13  Nikita Vasilyev  <nvasilyev@apple.com>
> 
>         REGRESSION (r188325): Web Inspector: Fix vertical spacing in CodeMirror
>diff --git a/Source/WebInspectorUI/UserInterface/Views/TimelineOverview.css b/Source/WebInspectorUI/UserInterface/Views/TimelineOverview.css
>index d9674d65e5e33623a95c5ac15680d38a2b674703..68e74ed02d880e2f285b17a0d721bb6b0749cfcb 100644
>--- a/Source/WebInspectorUI/UserInterface/Views/TimelineOverview.css
>+++ b/Source/WebInspectorUI/UserInterface/Views/TimelineOverview.css
>@@ -61,7 +61,7 @@
>     -webkit-transform: translateX(5px);
> }
> 
>-.timeline-overview.frames > .timeline-ruler > .markers,
>+.timeline-overview.frames > .timeline-ruler > .markers > .divider,
> .timeline-overview.frames > .timeline-ruler:not(.both-handles-clamped) > .shaded-area.right {
>     -webkit-transform: translateX(1px);
> }

Actions: View | Formatted Diff | Diff

Attachments on bug 147650: 258872 | 258910