Summary: | Web Inspector: Update MemoryTimelineView for new ruler height | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Matt Baker <mattbaker> | ||||||||||
Component: | Web Inspector | Assignee: | Matt Baker <mattbaker> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | bburg, commit-queue, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | WebKit Nightly Build | ||||||||||||
Hardware: | All | ||||||||||||
OS: | All | ||||||||||||
Bug Depends on: | 153034 | ||||||||||||
Bug Blocks: | |||||||||||||
Attachments: |
|
Description
Matt Baker
2016-02-04 17:45:04 PST
Created attachment 270710 [details]
[Patch] Proposed Fix
Created attachment 270711 [details]
[Image] Updated Memory timeline view
Comment on attachment 270710 [details] [Patch] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=270710&action=review > Source/WebInspectorUI/UserInterface/Views/MemoryTimelineView.css:59 > + padding-top: 2px; I still think a padding-left makes more sense. Previously this would have matched up with a title, but I got rid of that to reduce vertical space. What is the verdict here? Created attachment 272729 [details]
[Patch] Proposed Fix
Created attachment 272730 [details]
[Image] Updated view with decreased padding
Not sure how "Categories" is supposed to be left-aligned, so I left it alone.
Comment on attachment 272729 [details] [Patch] Proposed Fix Clearing flags on attachment: 272729 Committed r197493: <http://trac.webkit.org/changeset/197493> All reviewed patches have been landed. Closing bug. |