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

Preload types #22658

Merged
merged 4 commits into from
Mar 27, 2024
Merged

Preload types #22658

merged 4 commits into from
Mar 27, 2024

Conversation

tunetheweb
Copy link
Contributor

@tunetheweb tunetheweb commented Mar 19, 2024

Summary

Chrome is noted as only having partial support as not supporting certain types (added in #9578) but this is actually true of all browsers so incorrect to label Chrome as only having partial support. See this example test as evidence. In fact Firefox supports one less type (track) than Chrome (but raised).

Our choices are:

I've chosen to go with the second option as browsers seem fairly consistent here and I don't think marking all as partial support is particularly helpful. But open to changing that if you think we should do the other way.

I've also NOT marked Firefox as having partial support as I think the lack of track support is a minor thing. I have added a note to that effect though.

Test results and supporting details

See example here: https://resource-loading-examples.glitch.me/preload-types.html
(tested in Chrome 123, Firefox 123, and Safari 17.4)

Related issues

Content update here: mdn/content#32740
Also raised a spec discussion here: whatwg/html#8332 (comment)

And a caniuse request here: Fyrd/caniuse#7006

@github-actions github-actions bot added the data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML label Mar 19, 2024
Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

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

Thanks, I think another option would be to have sub features for all of the preload types. For now, this seems good to me, though.

@Elchi3 Elchi3 merged commit c43264a into mdn:main Mar 27, 2024
6 checks passed
@tunetheweb tunetheweb deleted the preload-types branch March 28, 2024 16:06
@tunetheweb
Copy link
Contributor Author

Thanks, I think another option would be to have sub features for all of the preload types. For now, this seems good to me, though.

Good idea. Raised #22728

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants