Re: [csswg-drafts] [css-color-4] Serialization needs to be updated for newly added xyz-d50, xyz-d65, oklab() and oklch() color spaces (#6834)

Thanks for noticing these areas where not all the consequences of "Resolve to add foo" got reflected in the spec!

> At the moment, I am assuming `oklab()` and `oklch()` serialize like `lab()` and `lch()` as specified in https://drafts.csswg.org/css-color/#serializing-lab-lch, but it should probably be made explicit.

> For `xyz-d50`, `xyz-d65`, I am assuming that they serialize like `color(xyz ...)` and also have a 16 bit minimum, but again, making it explicit would be good.

Yes. I just forgot to do those when I was adding in `oklab` and `oklch`.

> One change from the current spec I would like to propose is changing the serialization of `color(xyz ...)` to `color(xyz-d65 ...)` so that we don't have track that this is an alias in the engine past parsing, allow us to canonicalize xyz to xyz-d65 as early as possible.

That seems reasonable to me. Any counter-opinions?

-- 
GitHub Notification of comment by svgeesus
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/6834#issuecomment-981561423 using your GitHub account


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

Received on Monday, 29 November 2021 11:49:50 UTC