Skip to content

[css-color-4] Add display-p3-linear #12596 #12601

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

svgeesus
Copy link
Contributor

Draft for now until CSSWG resolution

@svgeesus svgeesus marked this pull request as draft August 13, 2025 15:44
@svgeesus svgeesus requested a review from annevk August 13, 2025 15:44
@svgeesus svgeesus linked an issue Aug 13, 2025 that may be closed by this pull request
@svgeesus
Copy link
Contributor Author

The sample code does not need to be updated as it already converts to and from display-p3-linear as part of converting to and from display-p3.

Copy link
Member

@annevk annevk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for putting this up! I'm not a color space expert, but I've done an editorial pass.

@@ -3,6 +3,7 @@ Title: CSS Color Adjustment Module Level 1
Shortname: css-color-adjust
Level: 1
Status: ED
Prepare for TR: no
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like it should be separated out?

@@ -4119,6 +4120,56 @@ The Predefined Display P3 Color Space: the ''display-p3'' keyword</h3>
2d.color.space.p3.toDataURL.with.putImageData.html
</wpt>

<h3 id="predefined-display-p3-linear">
The Predefined Linear-light Display P3 Color Space: the ''display-p3-linear'' keyword</h3>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would expect "Linear-Light" and also "Keyword" if you are going for title casing.

@@ -4714,7 +4765,7 @@ Color Space for Interpolation</h3>
Thus, different color spaces may be more appropriate for each interpolation use case.

* In some cases, the result of physically mixing two colored lights is desired.
In that case, the CIE XYZ or srgb-linear color space is appropriate, because they are linear in light intensity.
In that case, the CIE XYZ, display-p3-linear or srgb-linear color spaces are appropriate, because they are linear in light intensity.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should these be linked to their definition?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[css-color] Add display-p3-linear
2 participants