-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
report: add perf score explanation and link #9355
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -557,7 +557,7 @@ Util.numberDateLocale = 'en'; | |
*/ | ||
Util.UIStrings = { | ||
/** Disclaimer shown to users below the metric values (First Contentful Paint, Time to Interactive, etc) to warn them that the numbers they see will likely change slightly the next time they run Lighthouse. */ | ||
varianceDisclaimer: 'Values are estimated and may vary.', | ||
varianceDisclaimer: 'Values are estimated and may vary. The performance score is [based only on these metrics](https://github.com/GoogleChrome/lighthouse/blob/master/docs/scoring.md#how-are-the-scores-weighted).', | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @egsweeny do you we can should to the scoring doc in github or to https://developers.google.com/web/tools/lighthouse/v3/scoring ... in reality, both need updating, but google/WebFundamentals#7724 is already started. I kinda prefer the devsite page.. it's SEO is a bit stronger. And looks nice. :) |
||
/** Column heading label for the listing of opportunity audits. Each audit title represents an opportunity. There are only 2 columns, so no strict character limit. */ | ||
opportunityResourceColumnLabel: 'Opportunity', | ||
/** Column heading label for the estimated page load savings of opportunity audits. Estimated Savings is the total amount of time (in seconds) that Lighthouse computed could be reduced from the total page load time, if the suggested action is taken. There are only 2 columns, so no strict character limit. */ | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -45,6 +45,7 @@ | |
--color-gray-50: #FAFAFA; | ||
--color-gray-500: #9E9E9E; | ||
--color-gray-600: #757575; | ||
--color-gray-700: #616161; | ||
--color-gray-800: #424242; | ||
--color-gray-900: #212121; | ||
--color-gray: #000000; | ||
|
@@ -591,6 +592,9 @@ | |
color: var(--color-gray-600); | ||
margin: var(--section-padding-vertical) 0; | ||
} | ||
.lh-metrics__disclaimer a { | ||
color: var(--color-gray-700); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 800 was too dark compared to the text color |
||
} | ||
|
||
.lh-metric__description { | ||
display: none; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think these two were from a merge mismatch where
en-XL
was checked in but #9284 wasn't built on top of that commit, so these weren't updated. Shouldn't happen again (at least for any new PRs)