On Jun 22, 2009, at 6:02 PM, bulia byak wrote:
On Mon, Jun 22, 2009 at 8:48 PM, Felipe Sanches<felipe.sanches@...400...> wrote:
JonCruz,
I've been looking at the dropper-context.cpp code. It deals with sRGB colors only. I think that we should be able to pick an ICC color using the color picker. But there are some issues here:
My probably uninformed opinion is that, as these issues demonstrate, it is not a good idea to try to pick any non-RGB color at all. Inkscape rendering works in RGB, the screen works in RGB, and pretending that what we picked from the screen is not RGB is inherently clumsy. If you want to pass one object's ICC reference to another object, just use copy/paste style, color picker is a wrong tool for this.
I think bulia has it halfway here. I think we will need to allow for it since many users try to use it that way. However, we'll have to do it "right". What he pointed out will have to be addressed.
One thing is that as we move more towards using style (CSS styles), we'll have to preserve that in the workflow as much as possible also. When the picker is over our own canvas, it should eventually get CSS style name (when present) which in turn references and includes icc-color() on properties.
When it grabs something else (from another app's window) we could grab it as RGB, or we could possible find a current document stye that contained the same one. The latter would be a more complex piece to get right though.