Skip to content

[css-fonts-4] Clarify @font-palette-values at-rules as unavailable in shadow trees #6919

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

Open
drott opened this issue Dec 29, 2021 · 5 comments
Labels
css-fonts-4 Current Work

Comments

@drott
Copy link
Collaborator

drott commented Dec 29, 2021

Similar to #5167 I suggest we clarify what happens with @font-palette-value at-rules in shadow-trees, i.e. similar to that issue clarifying

"A @font-palette-values rule is invalid if it occurs in a stylesheet inside of a shadow tree, and must be ignored."

@drott
Copy link
Collaborator Author

drott commented Dec 29, 2021

Similarly, it may make sense to clarify if @font-palette-value rules are allowed to appear in @layer rules.

@xfq xfq added the css-fonts-4 Current Work label Dec 30, 2021
@drott
Copy link
Collaborator Author

drott commented Feb 9, 2022

Ping @litherum @svgeesus @jfkthame - do you have opinions on this? Can we clarify this question?

@jfkthame
Copy link
Contributor

jfkthame commented Feb 9, 2022

I guess this is fine for the time being. Will this include a "will likely change in the future" issue, though, similar to #5167? ISTM that in principle using @font-palette-value rules in shadow-trees should be perfectly reasonable, even if we're not able to allow it for now. (Or is there a reason to permanently exclude it?)

@tabatkins
Copy link
Member

My answer from 2020 stands - we only disallow insofar as impls are not yet following the rules in Scoping. Once an implementation supports those, then they must support these at-rules in shadow trees, following these rules.

@litherum
Copy link
Contributor

I don't know why font-palette-values should deserve any special treatment - it should follow whatever the behavior is for other at rules.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
css-fonts-4 Current Work
Projects
None yet
Development

No branches or pull requests

5 participants