Abdelrazak Younes <[EMAIL PROTECTED]> writes:
> > 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?
> Just for clarification, this would mean replacing x11name by this RGB
> value in ColorEntry:
> class ColorEntry {
> ColorEntry(
> string rgb_hexa = "#000000",
> string const guiname_ = "",
> string const latexname_ = "",
> string const lyxname_ = "");
Georg's solution may look complicated, but it's just a refactoring of existing,
working code. A real RGBColor class has real advantages, not least being type
safe. We (you :)) should strive to remove kludges, not add to them!
Further, a real Color lends itself to easy manipulation; the fact that we have
to separately define the colours of inset button and borders is a real ugliness!
À demain!
Angus
