Skip to content

[CSS Modules] Add non-constructable stylesheet to Open Issues#1264

Open
KurtCattiSchmidt wants to merge 1 commit intoMicrosoftEdge:mainfrom
KurtCattiSchmidt:nonconstructable-stylesheet-open
Open

[CSS Modules] Add non-constructable stylesheet to Open Issues#1264
KurtCattiSchmidt wants to merge 1 commit intoMicrosoftEdge:mainfrom
KurtCattiSchmidt:nonconstructable-stylesheet-open

Conversation

@KurtCattiSchmidt
Copy link
Contributor

This was suggested in w3ctag/design-reviews#1000 (comment).

Since we don't have a concrete answer for this scenario yet, this PR simply adds non-constructable stylesheets to the Open Issues list.

* What happens in scenarios that cross document boundaries, such as `Document.parseHTMLUnsafe`?
* How can developers check for and polyfill `shadowrootadoptedstylesheets`, given that the template element disappears from the DOM?
* Is it possible to define an intentional race between an async preload of an external stylesheet and a just-in-time definition of a declarative module and only apply the one that wins? This might not be possible due to the fact that specifiers are unique.
* Can this proposal work with non-constructable stylesheets? See https://github.com/w3c/csswg-drafts/issues/10013.
Copy link
Contributor

Choose a reason for hiding this comment

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

NIT: I'm a little hesitant to include an "open issue" for a potential independent change, but if so, I think "can" is an overly strong qualifier. This proposal can work with the existence of non-constructable stylesheets being used for adoptedstylesheets - it's just a question of how.

Suggested change
* Can this proposal work with non-constructable stylesheets? See https://github.com/w3c/csswg-drafts/issues/10013.
* How should this proposal work if non-constructable stylesheets are adopted for `adoptedstylesheets`? See https://github.com/w3c/csswg-drafts/issues/10013.

Copy link
Member

Choose a reason for hiding this comment

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

I like this rephrasing.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants