Re: [csswg-drafts] [css-color-4] `<hue> | none` is redundant (#7816)

> Nice, thanks for finding. Those test that `non` in legacy syntax is invalid. I don't think there are any that test that `none` in modern syntax is valid, though.

Oh, there are a ton of those too:
```
color-valid.html:test_valid_value("color", "rgb(none none none)", "rgb(0, 0, 0)");
color-valid.html:test_valid_value("color", "rgb(none none none / none)", "rgba(0, 0, 0, 0)");
color-valid.html:test_valid_value("color", "rgb(128 none none)", "rgb(128, 0, 0)");
color-valid.html:test_valid_value("color", "rgb(128 none none / none)", "rgba(128, 0, 0, 0)");
color-valid.html:test_valid_value("color", "rgb(none none none / .5)", "rgba(0, 0, 0, 0.5)");
color-valid.html:test_valid_value("color", "rgb(20% none none)", "rgb(51, 0, 0)");
color-valid.html:test_valid_value("color", "rgb(20% none none / none)", "rgba(51, 0, 0, 0)");
color-valid.html:test_valid_value("color", "rgb(none none none / 50%)", "rgba(0, 0, 0, 0.5)");
color-valid.html:test_valid_value("color", "rgba(none none none)", "rgb(0, 0, 0)");
color-valid.html:test_valid_value("color", "rgba(none none none / none)", "rgba(0, 0, 0, 0)");
color-valid.html:test_valid_value("color", "rgba(128 none none)", "rgb(128, 0, 0)");
color-valid.html:test_valid_value("color", "rgba(128 none none / none)", "rgba(128, 0, 0, 0)");
color-valid.html:test_valid_value("color", "rgba(none none none / .5)", "rgba(0, 0, 0, 0.5)");
color-valid.html:test_valid_value("color", "rgba(20% none none)", "rgb(51, 0, 0)");
color-valid.html:test_valid_value("color", "rgba(20% none none / none)", "rgba(51, 0, 0, 0)");
color-valid.html:test_valid_value("color", "rgba(none none none / 50%)", "rgba(0, 0, 0, 0.5)");
color-valid.html:test_valid_value("color", "hsl(none none none)", "rgb(0, 0, 0)");
color-valid.html:test_valid_value("color", "hsl(none none none / none)", "rgba(0, 0, 0, 0)");
color-valid.html:test_valid_value("color", "hsla(none none none)", "rgb(0, 0, 0)");
color-valid.html:test_valid_value("color", "hsla(none none none / none)", "rgba(0, 0, 0, 0)");
color-valid.html:test_valid_value("color", "hsl(120 none none)", "rgb(0, 0, 0)");
color-valid.html:test_valid_value("color", "hsl(120 80% none)", "rgb(0, 0, 0)");
color-valid.html:test_valid_value("color", "hsl(120 none 50%)", "rgb(128, 128, 128)");
color-valid.html:test_valid_value("color", "hsl(120 100% 50% / none)", "rgba(0, 255, 0, 0)");
color-valid.html:test_valid_value("color", "hsl(none 100% 50%)", "rgb(255, 0, 0)");
color-valid.html:test_valid_value("color", "hwb(none none none)", "rgb(255, 0, 0)");
color-valid.html:test_valid_value("color", "hwb(none none none / none)", "rgba(255, 0, 0, 0)");
color-valid.html:test_valid_value("color", "hwb(120 none none)", "rgb(0, 255, 0)");
color-valid.html:test_valid_value("color", "hwb(120 80% none)", "rgb(204, 255, 204)");
color-valid.html:test_valid_value("color", "hwb(120 none 50%)", "rgb(0, 128, 0)");
color-valid.html:test_valid_value("color", "hwb(120 30% 50% / none)", "rgba(77, 128, 77, 0)");
color-valid.html:test_valid_value("color", "hwb(none 100% 50% / none)", "rgba(170, 170, 170, 0)");
color-valid.html:    test_valid_value("color", `color(${colorSpace} none none none / none)`, `color(${colorSpace} none none none / none)`);
color-valid.html:    test_valid_value("color", `color(${colorSpace} none none none)`, `color(${colorSpace} none none none)`);
color-valid.html:    test_valid_value("color", `color(${colorSpace} 10% none none / none)`, `color(${colorSpace} 0.1 none none / none)`);
color-valid.html:    test_valid_value("color", `color(${colorSpace} none none none / 0.5)`, `color(${colorSpace} none none none / 0.5)`);
color-valid.html:    test_valid_value("color", `color(${colorSpace} 0 0 0 / none)`, `color(${colorSpace} 0 0 0 / none)`);
color-valid.html:    test_valid_value("color", `color(${colorSpace} none none none / none)`, `color(${resultColorSpace} none none none / none)`);
color-valid.html:    test_valid_value("color", `color(${colorSpace} none none none)`, `color(${resultColorSpace} none none none)`);
color-valid.html:    test_valid_value("color", `color(${colorSpace} 0.2 none none / none)`, `color(${resultColorSpace} 0.2 none none / none)`);
color-valid.html:    test_valid_value("color", `color(${colorSpace} none none none / 0.5)`, `color(${resultColorSpace} none none none / 0.5)`);
color-valid.html:    test_valid_value("color", `color(${colorSpace} 0 0 0 / none)`, `color(${resultColorSpace} 0 0 0 / none)`);
color-valid.html:    test_valid_value("color", `${colorSpace}(none none none / none)`, `${colorSpace}(none none none / none)`);
color-valid.html:    test_valid_value("color", `${colorSpace}(none none none)`, `${colorSpace}(none none none)`);
color-valid.html:    test_valid_value("color", `${colorSpace}(20 none none / none)`, `${colorSpace}(20 none none / none)`);
color-valid.html:    test_valid_value("color", `${colorSpace}(none none none / 0.5)`, `${colorSpace}(none none none / 0.5)`);
color-valid.html:    test_valid_value("color", `${colorSpace}(0 0 0 / none)`, `${colorSpace}(0 0 0 / none)`);
color-valid.html:    test_valid_value("color", `${colorSpace}(none none none / none)`, `${colorSpace}(none none none / none)`);
color-valid.html:    test_valid_value("color", `${colorSpace}(none none none)`, `${colorSpace}(none none none)`);
color-valid.html:    test_valid_value("color", `${colorSpace}(20 none none / none)`, `${colorSpace}(20 none none / none)`);
color-valid.html:    test_valid_value("color", `${colorSpace}(none none none / 0.5)`, `${colorSpace}(none none none / 0.5)`);
color-valid.html:    test_valid_value("color", `${colorSpace}(0 0 0 / none)`, `${colorSpace}(0 0 0 / none)`);
```

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


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

Received on Friday, 7 October 2022 04:10:42 UTC