-
Notifications
You must be signed in to change notification settings - Fork 711
[cssom-1] [css-cascade] CSSImportRule.sheet not being null conflicts with @import supports() #8608
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
Comments
@tabatkins @fantasai do you know if tweaking the OM spec to allow styleSheet to be nullable needs a resolution? |
CSSOM is CR-level, so yes, normative changes need WG resolution. (But should be quick and easy.) |
@tabatkins CSSOM is still WD-level, not CR-level. Sebastian |
…,webidl,smaug This doesn't match the CSSOM spec, but it seems fairly uncontroversial, and our WebIDL already has the nullable annotation, see: w3c/csswg-drafts#8608 This doesn't change behavior on its own, but it's needed for the other patches in this bug. Differential Revision: https://phabricator.services.mozilla.com/D172891
The CSS Working Group just discussed
The full IRC log of that discussion<fantasai> emilio: 2 specs conflicting with each other<fantasai> ... imported stylesheets cannot be null, but since we have supports() condition the UA is not required to fetch the stylesheet <fantasai> ... so we need CSSOM to allow it to be null <TabAtkins> +1 for the obvious fix <fantasai> +1 <fantasai> RESOLVED: Update CSSOM to allow null representation of imported style sheets |
https://drafts.csswg.org/css-cascade-5/#at-import:
But https://drafts.csswg.org/cssom-1/#dom-cssimportrule-stylesheet is not nullable.
I think the fix is obvious (just make it nullable). Not sure if that needs a resolution tho?
The text was updated successfully, but these errors were encountered: