Skip to content

Updates to modulepreload and preload to incorporate WHATWG changes for "style" and "json"#42333

Merged
chrisdavidmills merged 5 commits intomdn:mainfrom
KurtCattiSchmidt:modulepreload-mdn
Mar 4, 2026
Merged

Updates to modulepreload and preload to incorporate WHATWG changes for "style" and "json"#42333
chrisdavidmills merged 5 commits intomdn:mainfrom
KurtCattiSchmidt:modulepreload-mdn

Conversation

@KurtCattiSchmidt
Copy link
Contributor

@KurtCattiSchmidt KurtCattiSchmidt commented Dec 9, 2025

Description

Updates docs for preload and modulepreload per WHATWG changes in whatwg/html#11981

Motivation

This change aligns behavior with the current specs.

Additional details

Do not land until the WHATWG PR lands.

Related issues and pull requests

@KurtCattiSchmidt KurtCattiSchmidt requested a review from a team as a code owner December 9, 2025 22:52
@KurtCattiSchmidt KurtCattiSchmidt requested review from chrisdavidmills and removed request for a team December 9, 2025 22:52
@github-actions github-actions bot added Content:HTML Hypertext Markup Language docs size/s [PR only] 6-50 LoC changed labels Dec 9, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 10, 2025

Preview URLs (2 pages)

(comment last updated: 2026-03-04 08:27:33)

Copy link
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

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

@KurtCattiSchmidt Hi there, and thanks for the update. The changes seem reasonable as per the suggested spec change; I've given you a couple of minor suggestions to consider.

One thing though — it would make sense to hold off on this MDN change at least until the spec change is merged, so that MDN is consistent with the spec.

KurtCattiSchmidt and others added 2 commits December 10, 2025 16:12
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
@mathewhodson
Copy link
Contributor

mathewhodson commented Feb 1, 2026

Also as=style and as=json should be added to rel=modulepreload in https://github.com/mdn/browser-compat-data so they are shown in the compatibility table like they are in https://developer.mozilla.org/en-US/docs/Web/HTML/Reference/Attributes/rel/preload

@KurtCattiSchmidt
Copy link
Contributor Author

Also as=style and as=json should be added to rel=modulepreload in https://github.com/mdn/browser-compat-data so they are shown in the compatibility table like they are in https://developer.mozilla.org/en-US/docs/Web/HTML/Reference/Attributes/rel/preload

Thanks for the heads-up @mathewhodson! I have a PR up for this: mdn/browser-compat-data#29048

Once that lands, I can add the support table to this PR.

@KurtCattiSchmidt KurtCattiSchmidt requested a review from a team as a code owner March 4, 2026 00:27
@KurtCattiSchmidt
Copy link
Contributor Author

@mathewhodson - compat changes landed, so this is already up-to-date! Thanks for the reminder there

@chrisdavidmills - the spec change landed in the HTML5 spec in January, so this is good to land (see whatwg/html#11981)

Copy link
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

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

Yup, all good to go. Thanks a lot for your work on this, @KurtCattiSchmidt!

@chrisdavidmills chrisdavidmills merged commit f529ead into mdn:main Mar 4, 2026
8 checks passed
jdatapple pushed a commit to jdatapple/content that referenced this pull request Mar 6, 2026
…s for "style" and "json" (mdn#42333)

* Adding changes to link tag page

* Update files/en-us/web/html/reference/elements/link/index.md

Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>

* Update files/en-us/web/html/reference/elements/link/index.md

Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>

* Adding script to modulepreload as= section

* Removing extra and

---------

Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:HTML Hypertext Markup Language docs size/s [PR only] 6-50 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants