-
Notifications
You must be signed in to change notification settings - Fork 180
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
Support the p3 color space in the raster mark #2138
Comments
I would love that, also for "p3" color scales. |
Another potential problem (if I remember correctly from my last tentative), is that the canvas we create is then serialized into the SVG with Line 168 in 1d01e25
I'm not sure if we can keep its colorSpace as we do that. |
To close the loop, this fantastic notebook just published on the topic:
(thanks, @jwoLondon!) |
We don’t use d3-color for parsing anymore, as of #1454 (#1851). See Lines 514 to 527 in e50254d
So I think the only thing missing here is some way of specifying the colorSpace for the raster mark’s canvas. Retitling the issue accordingly. |
Ah thanks for reminding me @Fil. I like your proposed solution. How does it perform going through canvas? |
It sure takes a hit: on my machine rasterVaporP3 takes about 300ms, compared to rasterVapor 110ms. Still, it's not too awful, and it becomes an optimization problem. It will be fun to research once we think it's "correct". I feel it would be useful to apply this technique also in the srgb + CSS4 case — unless we get to a point where using fillRect is faster than editing the data matrix directly… but I doubt that there is any chance of that happening). Then we can say that css 4 colors are fully supported. |
Another interesting post on the topic, by @clhenrick: |
Enhancement request: Allow the raster mark to use CSS4 colour specifications and the P3 colour space.
Marks that generate SVG currently work with modern colour specifications, meaning they can specify colours using new colour spaces such as Oklch. For example, this works correctly:
Not only does this give access to improved perceptual colour spaces, but also allows deeply saturated colours outside the sRGB gamut but in the P3 gamut to be rendered correctly on displays that support P3 (with a graceful fallback to the nearest sRGB colour for those which don't).
However, the raster mark, which relies on Canvas, does not currently support either modern colour specification, nor the P3 colour space. The following renders a raster as black rather than the intended saturated red:
It would be great to have support for modern colour specs in Canvas.
As far as I can see, the current implementation creates an sRGB canvas rather than a p3-friendly one. I think that would just require
But handling CSS 4 colour specs looks a little more tricky as the implementation seems to rely on d3.color which currently does not handle the modern comma-free colour format. I note d3 issue #87 though.
The text was updated successfully, but these errors were encountered: