Re: [csswg-drafts] [css-shapes-1] Optional values shouldn't be serialized (#8695)

The CSS Working Group just discussed `[css-shapes-1] Optional values shouldn't be serialized`, and agreed to the following:

* `RESOLVED: We fix the example and tests based on it`
* `RESOLVED: Remove the part about serializing calc expressions from Shapes spec`

<details><summary>The full IRC log of that discussion</summary>
&lt;dael_> fantasai: The css shapes spec tries to spec serialization but does a couple of things that don't make sense. In an example it says omitting components shows some default values don't show in serialization.<br>
&lt;dael_> fantasai: You can omit @position so taht doesn't make sense. We have interop that we always serialize, but since you can omit you should allow to omit. Shortest serialization principle.<br>
&lt;dael_> fantasai: Second is some guidance about avoiding calc and calc transforms, but that conflicts with rules about serializing position in calc.<br>
&lt;dael_> fantasai: In this case, should defer to rules in values 4<br>
&lt;TabAtkins> strong +1 to making these consistent now<br>
&lt;dael_> astearns: This was written before the rules. Internt was always to be superceeded by calc. Happy to defer to existing text now<br>
&lt;dael_> astearns: First bit needs to stay in shapes b/c talks about specific shapes values, right?<br>
&lt;dael_> fantasai: I think we just need to correct the example and remove/fix the tests<br>
&lt;dael_> astearns: Any concerns about anybody relying on this? I don't think I have any<br>
&lt;dael_> astearns: Prop: We fix the example and tests based on it<br>
&lt;dael_> astearns: Obj?<br>
&lt;dael_> RESOLVED: We fix the example and tests based on it<br>
&lt;dael_> astearns: Prop: Remove the part about serializing calc expressions from Shapes spec<br>
&lt;dael_> astearns: Obj?<br>
&lt;dael_> RESOLVED: Remove the part about serializing calc expressions from Shapes spec<br>
&lt;dael_> astearns: Did you happen to look at the tests for serializing calc in Shapes? There may be some that need updates<br>
&lt;dael_> fantasai: I think there are tests, but I don't remember where they are<br>
&lt;dael_> astearns: Anything else on this one?<br>
</details>


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


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

Received on Wednesday, 2 August 2023 23:51:20 UTC