Georg Baum a écrit :
Am Dienstag, 21. März 2006 14:12 schrieb Angus Leeming:
There's an RGBColor class (and an HSVColor one too) in the XForms
frontend.
Please move 'em someplace useful.
Nice, I stumbled about RGBColor but thought it was an xforms thing.
The attached patch moves these classes to the core, and changes Branch to
store a RGBColor instead of a string. The consequence is that a branch
cannot have a "none" color anymore, and therefore some checks in the
frontend are not necessary anymore. LColor::background is used now
instead of "none". This is consistent with the current display of
branches with "none" color.
What do you think?
The patch looks good but I am wondering if this solution is not a bit
too complicated. Why not just define some const string in hexa and let
the frontend take care of the rest?
Abdel.
