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

Crux redesign #2731

Merged
merged 4 commits into from
Mar 7, 2023
Merged

Crux redesign #2731

merged 4 commits into from
Mar 7, 2023

Conversation

scottjehl
Copy link
Contributor

fixes #1721

First steps on redesigning our contextual crux metrics comparison. Can be viewed on my test server.

Impacted views:

  • result summary page
  • result details page (crux now only shows up for first view runs, not repeat views)
  • core web vitals page (now grouped separately from our intro metrics)

Preview in Results summary page (Test Run details page looks similar):
image

Preview in CWV page:
image

@scottjehl scottjehl requested a review from tkadlec March 2, 2023 22:38
@scottjehl scottjehl merged commit 5009c74 into master Mar 7, 2023
@scottjehl scottjehl deleted the crux branch March 7, 2023 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CrUX presentation needs to lead with a metric number and more clearly communicate differences from test run
1 participant