|Summary:||Web Inspector: records counter should be implemented in Timeline Panel.|
|Product:||WebKit||Reporter:||Ilya Tikhonovsky <loislo>|
|Component:||Web Inspector (Deprecated)||Assignee:||Nobody <webkit-unassigned>|
|Version:||528+ (Nightly build)|
Description Ilya Tikhonovsky 2010-03-27 12:06:20 PDT
As far as we have records filters it will be user friendly to have a counter of visible/captured records.
Comment 1 Ilya Tikhonovsky 2010-03-27 12:09:39 PDT
Created attachment 51835 [details] [patch] first iteration. has binary changes in localizedStrings.js
Comment 3 Timothy Hatcher 2010-03-27 12:35:52 PDT
It shouldn't be that big of font, maybe 1px smaller. Also not bold. It should have a 1px white text shadow on the bottom too.
Comment 4 Ilya Tikhonovsky 2010-03-27 12:56:33 PDT
Created attachment 51839 [details] Screenshot with changed font style
Comment 5 Timothy Hatcher 2010-03-27 12:57:26 PDT
Looks good with the new style.
Comment 6 Ilya Tikhonovsky 2010-03-27 13:00:07 PDT
Created attachment 51841 [details] [patch] second iteration with changed text style
Comment 7 Ilya Tikhonovsky 2010-03-29 12:02:55 PDT
Created attachment 51944 [details] [patch] fixed version Extended version of patch. 1) records count is not affected by collapsed states of the records; 2) records count is affected by type filters; 3) zero size events were not visible if they were last events in Timeline - fixed; 4) zero size event was filtered if it was the only event in Timeline - fixed; 5) type filter was applied only for the top level events - fixed.
Comment 8 Eric Seidel (no email) 2010-03-29 22:51:07 PDT
Comment on attachment 51944 [details] [patch] fixed version This is a git binary diff which uses delta diffs instead of literal diffs, unfortunately our scripts don't yet support delta diffs, so this will have to be either re-posted or landed by hand.