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

<link rel="prefetch"> _can_ interfere with loading higher priority resources so we should delay adding them #12850

Closed
KyleAMathews opened this issue Mar 25, 2019 · 1 comment
Labels
help wanted Issue with a clear description that the community can help with. type: maintenance An issue or pull request describing a change that isn't a bug, feature or documentation change

Comments

@KyleAMathews
Copy link
Contributor

From @yoavweiss in #12800 (comment)

I'd caution against heavily relying on prefetch's prioritization downgrade too much, as it doesn't always prevent contention with resources for the current page, especially if their discovery is delayed. It might be better to inject those prefetches only after you know that the current page is fully rendered and interactive.

We add prefetch elements here

// Add preload/prefetch <link>s for scripts.

We should instead insert a JSON blob into the HTML with these hrefs and add them as link elements once the page is rendered/interactive.

@wardpeet wardpeet added the type: maintenance An issue or pull request describing a change that isn't a bug, feature or documentation change label Mar 27, 2019
@KyleAMathews KyleAMathews added the help wanted Issue with a clear description that the community can help with. label May 23, 2019
@thall1961
Copy link

Do you still need help with this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Issue with a clear description that the community can help with. type: maintenance An issue or pull request describing a change that isn't a bug, feature or documentation change
Projects
None yet
Development

No branches or pull requests

6 participants