search

ftlabs / perf-widget

22 stars 2 forks source link

more pagespeed insights surfaced #61

Closed JakeChampion closed 8 years ago

JakeChampion commented 8 years ago

screen shot 2016-02-08 at 17 58 51

AdaRoseCannon commented 8 years ago

Looks good, but there are 3 performance subheads! Can google pagespeeds and performance be grouped together. i.e.:

Performance

Google Pageseed

JakeChampion commented 8 years ago

Yup, this pull request does not touch the UI as that is a different action item, it's added some more insights.

AdaRoseCannon commented 8 years ago

I don't really get the sql query so not entirely comfortable reviewing this. But it generally seems fine.