- From: Sam Atkins via GitHub <noreply@w3.org>
- Date: Tue, 19 Aug 2025 12:07:07 +0000
- To: public-houdini-archive@w3.org
AtkinsSJ has just created a new issue for https://github.com/w3c/css-houdini-drafts:
== [css-typed-om-1] "Serialize a CSSNumericValue" doesn't pass "nested" and "paren-less" to CSSMathValue ==
In the [algorithm for serializing CSSMathValue](https://drafts.css-houdini.org/css-typed-om-1/#serialize-a-cssmathvalue), it repeatedly ends up calling "serialzie a CSSNumericValue", because that's what CSSNumericArray contains. eg:
> 2. If this is a [CSSMathMin](https://drafts.css-houdini.org/css-typed-om-1/#cssmathmin) or [CSSMathMax](https://drafts.css-houdini.org/css-typed-om-1/#cssmathmax):
> 1. Append "min(" or "max(" to s, as appropriate.
> 2. [For each](https://infra.spec.whatwg.org/#list-iterate) arg in this’s [values](https://drafts.css-houdini.org/css-typed-om-1/#dom-cssmathmin-values) internal slot, serialize arg with nested and paren-less both true, and append the result to s, appending a ", " between successive values.
> 3. Append ")" to s and return s.
However, [serialize a CSSNumericValue](https://drafts.css-houdini.org/css-typed-om-1/#serialize-a-cssnumericvalue) doesn't make use of those arguments. It only accepts the minimum/maximum arguments that are relevant for CSSUnitValue. It needs to take the `nested` and `paren-less` arguments, and then pass them to "serialize a CSSMathValue".
Please view or discuss this issue at https://github.com/w3c/css-houdini-drafts/issues/1151 using your GitHub account
--
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config
Received on Tuesday, 19 August 2025 12:07:08 UTC