[css-conditional-5] Add 'none'-keywords to scroll-state() features by lilles · Pull Request #10874 · w3c/csswg-drafts (original) (raw)

This is largely reasonable. However, I took a slightly deeper look at the issues related to evaluating in a boolean context, and I realize that there's no general statement about features being usable in a boolean context for container features, like there is for media features.

It might be better to add a paragraph in the introduction to container features that repeats the first two paragraphs of https://drafts.csswg.org/mediaqueries-4/#mq-boolean-context (or refers to them). If you have that, then you don't need any mention of evaluation in a boolean context in the feature definitions at all.

And I think it's probably correct -- I'm guessing container features all evaluate in a boolean context just like media features do.

(On the other hand, I think it's ok to land this if you don't want to do that right now, but I think doing that should also be pretty straightforward.)