Re: [csswg-drafts] [css-contain-3] .conditionText and name vs query (#7190)

The CSS Working Group just discussed `[css-contain-3] .conditionText and name vs query`, and agreed to the following:

* `RESOLVED: Add .containerName and .containerQuery`

<details><summary>The full IRC log of that discussion</summary>
&lt;fantasai> Subtopic: [css-contain-3] .conditionText and name vs query<br>
&lt;fantasai> github: https://github.com/w3c/csswg-drafts/issues/7190<br>
&lt;fantasai> miriam: Currently we have .conditionText which is a single string<br>
&lt;fantasai> miriam: but includes 2 parts, a name for name of container and a query for the conditions to match against that container<br>
&lt;fantasai> miriam: so proposal is to keep .conditionText as the entire query<br>
&lt;fantasai> miriam: but add .containerName and .containerQuery so you can get the parts yourself<br>
&lt;fantasai> astearns: if we have the parts, why do we need the concatenated thing?<br>
&lt;fantasai> TabAtkins: to match the other conditional rules<br>
&lt;fantasai> heycam: it inherits from ConditionalRule, so need to say what that property returns<br>
&lt;fantasai> astearns: Anyone with opinions?<br>
&lt;TabAtkins> +1<br>
&lt;fantasai> +1 from fantasai<br>
&lt;fantasai> heycam: makes sense to me<br>
&lt;TabAtkins> (and emilio already implemented it, apparently)<br>
&lt;fantasai> heycam: just question, the one that returns the expression<br>
&lt;fantasai> heycam: that doesn't include the parentheses?<br>
&lt;fantasai> miriam: that would include potentially multiple expressions<br>
&lt;fantasai> heycam: was asking about outermost parens<br>
&lt;fantasai> miriam: they're not required<br>
&lt;fantasai> miriam: no need to include them there<br>
&lt;fantasai> heycam: ah, so parens are part of the expression, not extra syntax<br>
&lt;fantasai> miriam: right<br>
&lt;fantasai> astearns: objections?<br>
&lt;fantasai> miriam: @andruud already posted a PR, so we could resolve to merge the PR<br>
&lt;fantasai> astearns: Not seeing anything about parens<br>
&lt;fantasai> miriam: if they are included, not part of the .containerQuery?<br>
&lt;fantasai> astearns: need to know whether to expect the parens<br>
&lt;fantasai> miriam: could merge the PR and add that as well<br>
&lt;astearns> ack fantasai<br>
&lt;heycam> fantasai: the syntax is like for MQs<br>
&lt;heycam> fantasai: so there are potentially a redundant set of parens around the entire expression, but in lots of cases you'll have ( expr ) or ( expr )...<br>
&lt;heycam> fantasai: any parens there should appear there<br>
&lt;heycam> fantasai: if we want to simplify, we should do it all the way through<br>
&lt;fantasai> heycam: I guess I wasn't suggesting simplification<br>
&lt;fantasai> heycam: wasn't sure if outermost parens were part of @container rule syntax<br>
&lt;fantasai> heycam: but sounds like it's part of the expression<br>
&lt;fantasai> astearns: so it's just as specified, with whatever parens<br>
&lt;fantasai> miriam: yes<br>
&lt;fantasai> astearns: then we can resolve to accept this PR and add these attributes<br>
&lt;fantasai> RESOLVED: Add .containerName and .containerQuery<br>
</details>


-- 
GitHub Notification of comment by css-meeting-bot
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/7190#issuecomment-1240008814 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Wednesday, 7 September 2022 23:14:35 UTC