[patch] css parser, floats

Dirk Mueller mueller at kde.org
Sat Feb 28 14:44:37 GMT 2004


On Saturday 28 February 2004 03:26, Germain Garand wrote:

> 1) an overlook in the CSS parser which makes it occasionally skip a
> valid declaration if it's preceded by an invalid one
> -see attached testcase + css_parser.diff
> -real world example:  http://www.inforlandia.pt/
> ( => hover left menu "Catálogo" => dhtml menu get a transparent bg )

ugh, thanks. please commit. 

> 2) fix a problem with floating images: when they aren't loaded yet, the
> page is first layouted with temporary 10x10 pixmaps, but once the final
> image is loaded, blocks affected by the float aren't relayouted.
> Results in ugly text overlaps on images on a lot of news sites
> (see #71445)

Do you have a testcase for this? I consider it dangerous to compare 
floatBottom() with m_height while we're trying hard to calculate m_height 
itself. 


Dirk




More information about the kfm-devel mailing list