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

Update runtime settings UI, add to PSI #10962

Closed
nathanmax opened this issue Jun 15, 2020 · 3 comments · Fixed by #13125
Closed

Update runtime settings UI, add to PSI #10962

nathanmax opened this issue Jun 15, 2020 · 3 comments · Fixed by #13125
Assignees
Labels
needs-design P1.5 PSI/LR PageSpeed Insights and Lightrider report

Comments

@nathanmax
Copy link

Summary

Could you please state clearly what is the speed of the Network you Run your tests especially on the Google Insight Speed test page

@patrickhulce patrickhulce added PSI/LR PageSpeed Insights and Lightrider needs-design labels Jun 15, 2020
@patrickhulce
Copy link
Collaborator

Thanks for filing @nathanmax! Definitely weird that we don't say anywhere in PSI the throttling settings like the regular Lighthouse report.

@paulirish paulirish changed the title Please state the Network Speed Add runtime settings to PSI report UI Jun 15, 2020
@paulirish
Copy link
Member

Discussed and we had consensus to add the runtime setting block to the bottom of the PSI report. 👍

fwiw @nathanmax https://github.com/GoogleChrome/lighthouse/blob/master/docs/throttling.md has the details on the network throttling we use

@paulirish paulirish changed the title Add runtime settings to PSI report UI Update runtime settings UI, add to PSI Sep 25, 2021
@paulirish
Copy link
Member

I have a doc exploring the new design: https://docs.google.com/document/d/1W66PKNDdcyNYDHnDBPvplyLAUfozdgk9FAZ3pjGieKA/edit#

The mock:
image

but in the doc i explore the text and extra details..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-design P1.5 PSI/LR PageSpeed Insights and Lightrider report
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants