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

core: merge Details.Opportunity with Details.Table #14771

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

alexnj
Copy link
Member

@alexnj alexnj commented Feb 9, 2023

So I took a stab at merging Details.Opportunity with Details.Table by introducing an isOpportunity?: boolean attribute to differentiate opportunities from tables.

  • This breaks backward compatibility of LHRs. Old LHRs will be upgraded through lighthouse-compatibility.
  • OverallSavings{Ms|Bytes} are now called wasted{Ms|Bytes}, as table summaries had it before. They mean about the same thing so I find the naming reasonable — wasted ms and bytes are opportunities. This would break less in compatibility (vs. making the table property sound more positive in nature, which would break compatibility for tables as well).

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

Successfully merging this pull request may close these issues.

3 participants