Skip to content

[Flight] Fix preload as attribute for stylesheets - #34760

Merged
unstubbable merged 1 commit into
react:mainfrom
unstubbable:fix-stylesheet-preload
Oct 10, 2025
Merged

unstubbable merged 1 commit into
react:mainfrom
unstubbable:fix-stylesheet-preload

Conversation

@unstubbable

@unstubbable unstubbable commented Oct 7, 2025 •

Copy link
Copy Markdown
Collaborator

Follow-up to #34604. For a stylesheet, we need to render <link rel="preload" as="style" ...>, and not <link rel="preload" as="stylesheet" ...>. (ref)

fixes vercel/next.js#84569

Follow-up to react#34604. For a stylesheet, we need to render `<link
rel="preload" as="style" ...>`, and not `<link rel="preload"
as="stylesheet" ...>`. ([ref](https://developer.mozilla.org/en-US/docs/Web/HTML/Reference/Attributes/rel/preload#what_types_of_content_can_be_preloaded))

fixes vercel/next.js#84569
@meta-cla meta-cla Bot added the CLA Signed label Oct 7, 2025
@github-actions github-actions Bot added the React Core Team Opened by a member of the React Core Team label Oct 7, 2025
@react-sizebot

Copy link
Copy Markdown

Comparing: a4eb2df...ebaef91

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.js = 6.68 kB 6.68 kB = 1.83 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 604.98 kB 604.98 kB = 107.14 kB 107.14 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB = 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 663.95 kB 663.95 kB = 117.03 kB 117.03 kB
facebook-www/ReactDOM-prod.classic.js = 687.81 kB 687.81 kB = 121.07 kB 121.07 kB
facebook-www/ReactDOM-prod.modern.js = 678.24 kB 678.24 kB = 119.42 kB 119.42 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against ebaef91

@unstubbable
unstubbable requested a review from eps1lon October 9, 2025 18:46
@unstubbable
unstubbable merged commit d446597 into react:main Oct 10, 2025
246 of 247 checks passed
@unstubbable
unstubbable deleted the fix-stylesheet-preload branch October 10, 2025 19:41
yannvr added a commit to hyperdrift-io/own-stack that referenced this pull request Oct 8, 2026
React 19.3 carries react/react#34760, which ends the 60-second blank
page when a dynamic page throws not-found mid-stream under a layout that
imports CSS. Waku's peer range has required it since rc.1.

No code changes: this app has no _root.tsx, so the default root now
renders charset and viewport (rc.2), and it uses file routes only, so
rc.3's defineRouter redesign does not touch it. The waku pin stays exact.

Checked: type-check, build, tests, and the production build serving
/ 200, /health 200 and an unknown path 404.
yannvr added a commit to hyperdrift-io/stillness that referenced this pull request Oct 8, 2026
React 19.3 is Waku's peer range since rc.1 and carries
react/react#34760 (no 60 s blank page when a page throws not-found
mid-stream under a layout that imports CSS).

rc.2 moved charset and viewport out of the framework head into the
default root, so the custom _root.tsx renders both now. File routes
only, so rc.3's defineRouter redesign does not apply.

Checked: type-check, build, the production build serving / 200,
/llms.txt 200 and an unknown path 404, and the home page hydrating in
the browser with both meta tags present once.
yannvr added a commit to hyperdrift-io/unanswered that referenced this pull request Oct 8, 2026
React 19.3 is Waku's peer range since rc.1 and carries
react/react#34760 (no 60 s blank page when a page throws not-found
mid-stream under a layout that imports CSS).

No code changes: no custom _root.tsx, so the default root renders
charset and viewport (rc.2), and file routes only, so rc.3's
defineRouter redesign does not apply. Installed past the 3-day
min-release-age guard: waku rc.3 is two days old, has provenance
attestations and no OSV advisory.

Checked: type-check, build, tests (6 pass), and the production build
serving / 200, /health 200, /llms.txt 200 and an unknown path 404.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Link preload must have a valid as

3 participants