Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Re-Organize Page Metrics Table #14

Open
hariramachandra opened this issue Jan 5, 2013 · 4 comments
Open

Re-Organize Page Metrics Table #14

hariramachandra opened this issue Jan 5, 2013 · 4 comments

Comments

@hariramachandra
Copy link
Contributor

What : We need to re-organize the contents of page metrics table based on page timing and non-page timing metrics

Why : Its convenient to have metrics based on timing under one table as it makes it easy to read the report and helps in automated comparison between mobster reports

How : I propose we have two tables under page metrics (1) Page Timing Metrics (2) Page Metrics

(1) Page Timing Metrics table :

DOMContent Load Time [ available]
OnLoad Time [ available]
Total CSS Time [ available]

(2) Page Metrics Table :

Number of Style Recalculates [ available ]
Number of Paints [ available ]
Total Page Weight [ TBD ] - details in #11
Download Time [TBD ] - details in
Total number of Images [ TBD] - no tickets opened yet

@ghost ghost assigned armanhb Jan 5, 2013
@armanhb
Copy link
Contributor

armanhb commented Jan 8, 2013

Is this what you had in mind (except for the missing # of images)?

new_report

@hariramachandra
Copy link
Contributor Author

Arman,

We reviewed the page and looks good. Please commit. Thanks for implementing the feature in a short time

thanks
Hari

@armanhb
Copy link
Contributor

armanhb commented Jan 8, 2013

number of images will be added in another pull request

@hariramachandra
Copy link
Contributor Author

great! thanks Arman

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants