251152 2023-01-25 08:15:00 -0800 color() function parsing incorrectly parses missing components 2023-03-24 14:34:58 -0700 1 1 1 Unclassified WebKit CSS Other Unspecified Unspecified RESOLVED FIXED https://wpt.fyi/results/css/css-color/parsing/color-invalid-color-function.html?label=master&label=experimental&product=chrome&product=firefox&product=safari&aligned&view=interop&q=label%3Ainterop-2022-color%20or%20label%3Ainterop-2023-color BrowserCompat, GoodFirstBug, InRadar P2 Normal --- 1 emilio ntim chris heycam karlcow koivisto m_dubet ntim sam webkit-bug-importer oldest_to_newest 1928143 0 emilio 2023-01-25 08:15:00 -0800 color(srgb / 0.5) seems to parse on safari with the components zeroed out, but that seems wrong per spec. https://w3c.github.io/csswg-drafts/css-color-4/#funcdef-color only says the alpha is optional. 1928347 1 emilio 2023-01-25 17:19:22 -0800 https://bugs.chromium.org/p/chromium/issues/detail?id=1410200 is the relevant chromium bug, for reference. I found this while reviewing patches to implement this in Firefox. 1928359 2 karlcow 2023-01-25 18:19:49 -0800 After: document.body.style.color = color(srgb / 0.5) document.body.style.color WebKit 'color(srgb 0 0 0 / 0.5)' Gecko '' Blink 'color(srgb 0 0 0 / 0.5)' 1928512 3 webkit-bug-importer 2023-01-25 20:43:59 -0800 <rdar://problem/104679823> 1928606 4 chris 2023-01-26 07:58:31 -0800 This used to be correct per spec, long ago when color() also did custom color spaces and it was reasonable to omit some components and have them auto-filled with zero. Now custom color spaces are in CSS Color 5, and clearly distinguished with a dashed-ident; while predefined color spaces are in CSS Color 4 and are wither an RGB space or an XYZ space. Both take 3 components, as the grammar makes clear. There was some leftover prose about variable number of components, not up to date with the grammar change, which I just corrected. 1943813 5 ntim 2023-03-24 11:51:31 -0700 Pull request: https://github.com/WebKit/WebKit/pull/11940 1943856 6 ews-feeder 2023-03-24 14:34:49 -0700 Committed 262098@main (77b199a5f044): <https://commits.webkit.org/262098@main> Reviewed commits have been landed. Closing PR #11940 and removing active labels.