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

Make render-blocking requests for css and js more reliable in browsers that do not provide that information #2217

Merged
merged 3 commits into from
Aug 4, 2022

Conversation

scottjehl
Copy link
Contributor

This adds and utilizes a new custom metric that collects scripts and links in the page that have render-blocking qualities.

This list is not a guarantee to contain render-blockers, as these may have been added dynamically by the time this runs. Still, we can use this list to compare in browsers that don't send a render-blocking flag on requests, so we can be more confident that a request that arrived before start render was actually render-blocking. Browsers such as chromium currently give us that information directly so we do not need this in those browsers.

…ck render.

This is not a guarantee, as these may have been added dynamically by the time this runs. Still, we can use this list to compare in browsers that don't send a render-blocking flag on requests, so we can be more confident that a request that arrived before start render was actually render-blocking. Browsers such as chromium currently give us that information directly so we do not need this in those browsers.
@scottjehl scottjehl requested a review from tkadlec August 2, 2022 15:29
@stoyan
Copy link
Contributor

stoyan commented Aug 2, 2022

nit: could you run composer format:prettier? I think it will have some opinions of quotes and such

Copy link
Contributor

@tkadlec tkadlec left a comment

Choose a reason for hiding this comment

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

Think this one is looking pretty good, @scottjehl!

@tkadlec tkadlec merged commit bb0d911 into master Aug 4, 2022
@scottjehl scottjehl deleted the 2216 branch December 2, 2022 15:12
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.

None yet

3 participants