-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[ColorWidget] Add CMYK support #57361
Conversation
@troopa81 There's a fair amount of wasted space in the dialog caused by putting the combo box in the top left corner. Could we try moving it here instead? How does this look in the vertical orientation mode used in the styling dock color buttons too? Can you share a screenshot showing that? |
Question: is the use of rounded integer 0-255 values an issue when using CMYK colors? It's been on my wishlist for a while to revisit these widgets and make them return qreal values for the color components, so that we aren't losing information when a color is changed via the widget and are better equipped to handle higher precision color models in future (since these are already supported by both Qt and our color reading/writing code, it's really only the color widget itself holding us back). |
As much as RGB color I think. Do you have a specific issue in mind ? or just the lack of precision when you use the color wheel widget for instance |
66b738f
to
8e8179a
Compare
Way better indeed. Done and updated video accordingly |
One reason to use CMYK is so that you can hit very specific spot colours used by your printers. This allows prints to get very high DPI at the tones you specify, because they don't need to use screens to dither various colours together. If I can't specify the exact percentages from my printer's catalogue of spot colours, and have them propagate all the way through to a CMYK-colourspace PDF put out by the Layout Manager, then the result will be a very expensive print of a map with degraded quality. |
@nyalldawson I remoed the warning message @spacehobo Good point, I'll try to deal with this in another PR if I have the time/budget. |
A new setting activeCmykComponent has been introduced, while the old one activeComponent refers now only to the Rgb/Hsv part
8aceb92
to
bb97a02
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one little change left
This pull request has been tagged for the changelog.
You can edit the description. Format available for credits
Thank you! |
Usually CMYK values are given as percentages of the maximum amount of ink for the specific print. At least here in Germany. So if I get values, I need to convert them from percent to integer values. Otherwise, the colours become too light. See for example the table starting on page 11 of this pdf: Anhang_zu_den_Richtlinen_für_die_Forsteinrichtung_im_Körperschaftswald.PDF https://www.gesetze-bayern.de/Content/Document/BayVwV257553-78 |
That makes sense. GIMP displays it in percentage (CMYK in its last version) https://librearts.org/2022/08/gimp-2-99-12-cmyk/gimp-2-99-12-sample-points.webp Krita offers to switch between % and 0-255 range Not sure this is relevant to display CMYK in 0-255, so I'll propose a modification to display it always in percent. |
I just double checked with the "Swiss World Atlas" team (the printed school atlas) - and they confirmed that they enter values in percentages (usually 5% step increments). |
@andreasneumann thanks for the confirmation. Let's go for percent so! |
@troopa81 a couple of thoughts:
|
I was afraid you said so :) Why not, but I'm not sure exactly what is the avantage of selecting CMYK in raw value, or RGB in percent ?
Yes, It's in my todo list |
I'm unsure if there's any valid use case for setting CMYK as raw values, and I'm not opposed to just showing in percent always. (We've always shown HSV/alpha values as percent and not the raw values and there's been no requests in decades to expose these as raw values!) RGB as percent is important if we remove the forced integer rounding... and I'd argue that for many users setting a value to 50% percent is a much more natural choice then "127.5". But I don't want to add GUI work for you for my wants, so I'm happy to add this toggle myself after we get the non-int rounding in place 👍 |
My primary goal for CMYK colours is to be compatible with printers' "spot colours" when generating a PDF for submission. All printers I'm aware of use percentages (although they don't show the |
@spacehobo This has been taken care in #58376 so now CMYK color are edited in percent, not raw values. |
This is the first PR regarding CMYK QEP
Adds CMYK support in color widget so we can select CMYK color.
A new setting activeCmykComponent has been introduced, while the old one activeComponent refers now only to the Rgb/Hsv part
Funded by Métropôle de Bordeaux