Bug 158100

Summary: Web Inspector: memory category timelines would look better with ruler tick-aligned guides
Product: WebKit Reporter: Blaze Burg <bburg>
Component: Web InspectorAssignee: Devin Rousso <hi>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, graouts, hi, inspector-bugzilla-changes, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
[MOVIE] timelines don't do anything
none
Patch
none
[Image] After Patch is applied none

Blaze Burg
Reported 2016-05-25 16:43:59 PDT
Created attachment 279837 [details] [MOVIE] timelines don't do anything We have these in the timeline overview, but not below. I think we should have them in both places. It is weird if the category timelines are flat (same value) and don't change at all visually when the selected time changes. It almost looks like the timelines are stuck. See attached movie.
Attachments
[MOVIE] timelines don't do anything (5.42 MB, video/quicktime)
2016-05-25 16:43 PDT, Blaze Burg 		no flags
Details
Patch (2.60 KB, patch)
2017-03-16 20:00 PDT, Devin Rousso 		no flags
DetailsFormatted DiffDiff
[Image] After Patch is applied (257.69 KB, image/png)
2017-03-16 20:01 PDT, Devin Rousso 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Blaze Burg
Comment 1 2016-05-27 07:20:04 PDT
Ping for import.
Radar WebKit Bug Importer
Comment 2 2016-05-27 07:24:21 PDT
<rdar://problem/26516722>
Devin Rousso
Comment 3 2017-03-16 20:00:54 PDT
Created attachment 304741 [details] Patch
Devin Rousso
Comment 4 2017-03-16 20:01:14 PDT
Created attachment 304742 [details] [Image] After Patch is applied
Joseph Pecoraro
Comment 5 2017-03-16 20:12:52 PDT
Comment on attachment 304741 [details] Patch r=me Sweet! Looks great
WebKit Commit Bot
Comment 6 2017-03-16 22:29:54 PDT
Comment on attachment 304741 [details] Patch Clearing flags on attachment: 304741 Committed r214099: <http://trac.webkit.org/changeset/214099>
WebKit Commit Bot
Comment 7 2017-03-16 22:29:59 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.