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

performance-impl: start tracking lcp element #35123

Merged
merged 6 commits into from
Sep 18, 2021

Conversation

samouri
Copy link
Member

@samouri samouri commented Jul 2, 2021

summary

Creates a new csi beacon lcpt (Largest Contentful Paint Type) for tracking which type of element caused the LCP.

@samouri samouri self-assigned this Sep 17, 2021
@samouri samouri marked this pull request as ready for review September 17, 2021 19:43
@amp-owners-bot
Copy link

Hey @jridgewell! These files were changed:

src/core/constants/enums.js

src/service/performance-impl.js Outdated Show resolved Hide resolved
samouri and others added 2 commits September 17, 2021 15:47
Co-authored-by: Justin Ridgewell <justin@ridgewell.name>
Co-authored-by: Justin Ridgewell <justin@ridgewell.name>
@jridgewell jridgewell enabled auto-merge (squash) September 18, 2021 02:07
@jridgewell jridgewell merged commit 71f17a6 into ampproject:main Sep 18, 2021
@samouri samouri deleted the perf-lcp-type branch September 19, 2021 14:30
yotam-g pushed a commit to yotam-g/amphtml that referenced this pull request Sep 22, 2021
* performance-impl: start tracking lcp element

* tagName --> nodeName

* include video

* Update src/service/performance-impl.js

Co-authored-by: Justin Ridgewell <justin@ridgewell.name>

* Revert "Update src/service/performance-impl.js"

This reverts commit 8cb0f4a.

* Update src/service/performance-impl.js

Co-authored-by: Justin Ridgewell <justin@ridgewell.name>

Co-authored-by: Justin Ridgewell <justin@ridgewell.name>
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