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

Should use "fetch a CSS style sheet" from CSSOM #968

Open
Ms2ger opened this issue Mar 30, 2016 · 4 comments
Open

Should use "fetch a CSS style sheet" from CSSOM #968

Ms2ger opened this issue Mar 30, 2016 · 4 comments

Comments

@Ms2ger
Copy link
Member

Ms2ger commented Mar 30, 2016

@annevk
Copy link
Member

annevk commented Jun 14, 2019

The issue in the HTML Standard says this would replace "default fetch and process the linked resource", but I'm not sure how good that is given how much common infrastructure it covers, such as referrer policy and integrity.

@domenic
Copy link
Member

domenic commented Jun 14, 2019

Yes, these days I'd kind of prefer to invert the model. Especially after @domfarolino's work revamping link fetching to be more hookable, I think more likely we could define "linked resource fetch setup steps" for stylesheets, and keep it all centralized in HTML.

@zcorpan zcorpan removed their assignment Jun 14, 2019
@annevk
Copy link
Member

annevk commented Jan 7, 2020

I noticed that the CSSOM variant does not check for the response's status to be 2xx.

@noamr
Copy link
Contributor

noamr commented Oct 6, 2021

Yes, these days I'd kind of prefer to invert the model. Especially after @domfarolino's work revamping link fetching to be more hookable, I think more likely we could define "linked resource fetch setup steps" for stylesheets, and keep it all centralized in HTML.

I tend to agree. To make the CSS variant variable it would need to copy or import things from HTML such as "potentail-CORS" stuff which should have knowledge about crossorigin attributes of link elements - an HTML concern rather than CSS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
5 participants