Skip to content

feat: Report Web Vitals metrics on pageload transactions #2651

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

Closed
wants to merge 1 commit into from

Conversation

rhcarvalho
Copy link
Contributor

Web Vitals is an initiative by Google to provide unified guidance for
quality signals that are essential to delivering a great user experience
on the web.

https://web.dev/vitals/

Instead of a manual implementation targeting the performance API
directly, this commit adds metrics by using the web-vitals library.

Using web-vitals allows us to easily report on all supported metrics at
the cost of ~1kb in the minified @sentry/apm bundle.

Core Web Vitals

  • Cumulative Layout Shift (CLS)
  • First Input Delay (FID)
  • Largest Contentful Paint (LCP)

Other Web Vitals

  • First Contentful Paint (FCP)
  • Time to First Byte (TTFB)

@rhcarvalho
Copy link
Contributor Author

This is a PoC to compare with #2624.

@kamilogorek
Copy link
Contributor

Keep in mind that it adds 1.4kB gzipped. On the other hand if we plan to ship this functionality anyway, it'd better to have it standardized.

Web Vitals is an initiative by Google to provide unified guidance for
quality signals that are essential to delivering a great user experience
on the web.

https://web.dev/vitals/

Instead of a manual implementation targeting the performance API
directly, this commit adds metrics by using the web-vitals library.

Using web-vitals allows us to easily report on all supported metrics at
the cost of ~1kb in the minified @sentry/apm bundle.

Core Web Vitals
- Cumulative Layout Shift (CLS)
- First Input Delay (FID)
- Largest Contentful Paint (LCP)

Other Web Vitals
- First Contentful Paint (FCP)
- Time to First Byte (TTFB)
@rhcarvalho rhcarvalho force-pushed the rhcarvalho/web-vitals branch from 9aa8bb5 to a88f5b4 Compare June 8, 2020 12:04
@rhcarvalho
Copy link
Contributor Author

#2624 is merged. This serves as inspiration if we decide to add all the other metrics later.

@rhcarvalho rhcarvalho closed this Jun 8, 2020
@rhcarvalho rhcarvalho deleted the rhcarvalho/web-vitals branch June 8, 2020 12:23
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.

2 participants