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.

Sign up for GitHub

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

Jump to bottom

Image component preload prioritizes image above CSS #18416

Closed
atcastle opened this issue Oct 28, 2020 · 3 comments · Fixed by #18864
Closed

Image component preload prioritizes image above CSS #18416

atcastle opened this issue Oct 28, 2020 · 3 comments · Fixed by #18864
Assignees
@atcastle
Milestone
iteration 15

Comments

@atcastle
Copy link
Collaborator

Bug report

Describe the bug

The image component, when used with the priority attribute places the higher than the preload tag for CSS. This can cause a large delay in FCP.

To Reproduce

See any page using a priority image, such as https://demo.vercel.store/

Expected behavior

The image preload should appear between the css preloads and the javascript preloads
@okbel okbel mentioned this issue Oct 28, 2020
Closed
@Timer Timer added this to the iteration 11 milestone Oct 29, 2020
@Timer Timer modified the milestones: iteration 11, iteration 12 Oct 30, 2020
@atcastle atcastle mentioned this issue Nov 5, 2020
Merged
@Timer Timer modified the milestones: iteration 12, 10.x.x Nov 12, 2020
@Timer
Copy link
Member

Timer commented Nov 12, 2020

Related #19118

@Timer Timer modified the milestones: 10.x.x, iteration 12 Nov 15, 2020
@Timer
Copy link
Member

Timer commented Nov 15, 2020

Removed preloading for now as it slows down non-Chrome devices.

@Timer Timer closed this as completed Nov 15, 2020
@Timer Timer modified the milestones: iteration 12, iteration 15 Dec 30, 2020
@Timer Timer reopened this Dec 30, 2020
@balazsorban44
Copy link
Member

This issue has been automatically locked due to no recent activity. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you.

@vercel vercel locked as resolved and limited conversation to collaborators Jan 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@atcastle atcastle

Labels
None yet
Projects
None yet
Milestone
iteration 15
Development

Successfully merging a pull request may close this issue.

Move CSS Preloads to top of head at document render azukaru/next.js
3 participants
@Timer @atcastle @balazsorban44