Fwd: [patch] Re: kdelibs/khtml/dom

[Rob Kaper]

On Thu, Nov 07, 2002 at 10:36:57AM +0100, David Faure wrote:
> > Apparently kfm-devel is the better one for this. Was QColor
> > DOM::RGBColor::color() removed recently? KSVG seems to depend on that. It
> > doesn't need a real QColor though, but an accessor to the QRgb was necessary
> > for me to make it compile and run.
> 
> IMHO QColor color() should be readded too (possibly in addition to rgbColor(),
> for source and binary compatibility.

http://kdewebcvs.pandmservices.com/cgi-bin/cvsweb.cgi/kdelibs/khtml/dom/css_value.cpp
(1.20)

Hm, apparently Dirk changed it to fix a bug.. I'll readd color() and
RBGColor(QColor) when I get home tonight, to keep BC and SC. And we won't
need the new accessor from my patch..

I do not know if the use of color() can trigger a bug like "wrong table
colors when external stylesheets are involved" in KSVG, but I'll leave that
up to the KSVG developers.

Rob
-- 
Rob Kaper     | Gimme some love, gimme some skin,
cap at capsi.com | if we ain't got that then we ain't got much
www.capsi.com | and we ain't got nothing, nothing! -- "Nothing" by A
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20021107/995419e2/attachment.sig>