[Patch] marquee merge

Leo Savernik l.savernik at aon.at
Sat Jun 12 18:15:48 CEST 2004


Hello,

I finally merged marquee from Safari and it's working perfectly (i. e. it's 
exactly as annoying as in IE ;-) ).

I also fixed a bug in HTMLElementImpl::addCSSLength, where it ignored the 
numOnly parameter, thus converting the time data into px-Units (WebCore 
doesn't suffer from this, as it got rid of this parameter somehow).

Please review.

Note that there are severe problems wrt the clip rect in layer painting code. 
Whenever the whole page is refreshed, parts of the layer contents are drawn 
below and above of it (despite of hasOverflowClip). This is *not* caused by 
the marquee patch, it has existed before, and is also visible on 
overflow:scroll layers. I have spent the whole day tracking down the bug, but 
to no avail. 

mfg
	Leo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: khtml_marquee_2a.diff
Type: text/x-diff
Size: 66994 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/khtml-devel/attachments/20040612/81f2bd1e/khtml_marquee_2a-0001.bin


More information about the Khtml-devel mailing list