Version: 4.4.00 (KDE 4.4.0) (using 4.4.00 (KDE 4.4.0), Kubuntu packages)
OS: Linux (i686) release 2.6.32-13-generic
This page fails to render correctly in Konqueror 4.4.0:
the page correctly renders in 3.5.10 yet also fails in 4.3.5. There is nothing special about this page, it's just one that Konqueror fails to render now. The failure I see is in layout, all the text and images are overlayed rather then in separate columns.
That's a table layout problem.
At some point in the code there is the following definition:
<td align=left valign=top width=138 colspan=0 rowspan=0...
There was a fix about span=0 (r963656) some times ago that's the causes of the regression.
Just reverting r963656 makes the site looks better but, seems to me, we loose the right column menu (i have to test better that)
A local copy of the site (saved with FF) looks fine on Konq because unlike the online copy FF replaces (colspan=0 rowspan=0) with (colspan=1 rowspan=1)
> the right column menu (i have to test better that)
read: left column menu, sorry
Great, thanks for the reduction! so the page is not HTML 4.01 compliant.
I suppose we could wontfix it, but OTOH, colspan/rowspan=zero is ill-implemented in most UAs.
Opera has it only for colspan, Firefox has nice implementation for both but in standard/almost-standard mode only, MSIE doesn't implement that AFAIK, nor does Safari.
So the best solution here is to act similarly has firefox.
Not to drop in too much, but we got some nasty regressions due to colspan/rowspan=0 support which you may want to look at while in neighborhood --- bug #204297
Created attachment 40889 [details]
patch... (also containing a fixlet for #204297 crash)
@maksim, yes there are many things to fix in the implementation... I think Carlos got confused with the col/effCol thing (can't blame him, it's viciously complicating matters)... the code trying to read the colgroup/col settings is also half broken - I'll try to fix all that properly in a not too distant future.
In the meantime, avoiding to hit the impl too hard with tables that weren't meant for it is a start (e.g. for #204297, the table asked for colspan="" which should be translated to colspan="1", not zero)
SVN commit 1094429 by ggarand:
.rowspan="" should map to rowspan=1, not rowspan=0
avoids a crash on xmpp.org pages
.only actually enables the HTML 4.01 meaning of colspan/rowspan zero in
standard/almost-standard mode matching Gecko engines
M +10 -6 html_tableimpl.cpp
WebSVN link: http://websvn.kde.org/?view=rev&revision=1094429