Skip to content

Add informative note about how PaintTimingMixin works#116

Merged
noamr merged 1 commit intomainfrom
blurby
Sep 15, 2025
Merged

Add informative note about how PaintTimingMixin works#116
noamr merged 1 commit intomainfrom
blurby

Conversation

@noamr
Copy link
Contributor

@noamr noamr commented Sep 15, 2025

Closes #115
See also w3c/largest-contentful-paint/145


Preview | Diff

Closes #115
See also w3c/largest-contentful-paint/145
@noamr noamr requested a review from yoavweiss September 15, 2025 09:10
@noamr
Copy link
Contributor Author

noamr commented Sep 15, 2025

cc @smfr

Copy link

@mmocny mmocny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know you mentioned back links aren't common, but I think this section, which provides a larger context, is quite useful. Somewhat at risk of going stale over time but it's worth it.

Thanks.

@noamr noamr merged commit 8d65e0d into main Sep 15, 2025
2 checks passed
github-actions bot added a commit that referenced this pull request Sep 15, 2025
SHA: 8d65e0d
Reason: push, by noamr

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

The spec never says when "Mark paint timing" is called

2 participants