190493 - columns that dont are the origin of a cell dont get a width assigned in layout strategy.

Status: RESOLVED WORKSFORME

(Core :: Layout: Tables, defect)

Description

[Dennis Haney]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2.1) Gecko/20021130
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.2.1) Gecko/20021130

I would expect these two to have the same layout, however the one with colspan=2
in the last cell reverts my <col span=4 width="1*"/> to <col span=3 width="1*"/>


Reproducible: Always

Steps to Reproduce:
1.
2.
3.





<table border=1 width=600px style="background-color:lightgray">
<caption>with two td's</caption>
<colgroup><col span=4 width="1*"/></colgroup>
<tr align=center><td colspan=4>... ITL</td></tr>
<tr align=center><td>Flags</td><td>#Tables</td><td colspan=2>Number of
rows</td></tr>
<tr align=center><td colspan=2>End of Free Space</td><td
colspan=2>Available Free Space</td></tr>
<tr align=center><td colspan=2>Total Free Space</td><td>-</td><td>-</td></tr>
</table>



<table border=1 width=600px style="background-color:lightgray">
<caption>With colspan=2</caption>
<colgroup><col span=4 width="1*"/></colgroup>
<tr align=center><td colspan=4>... ITL</td></tr>
<tr align=center><td>Flags</td><td>#Tables</td><td colspan=2>Number of
rows</td></tr>
<tr align=center><td colspan=2>End of Free Space</td><td
colspan=2>Available Free Space</td></tr>
<tr align=center><td colspan=2>Total Free Space</td><td colspan=2>--</td></tr>
</table>

Comment 1

[Dennis Haney]

Attachment: same just put in a file

Comment 2

[Dennis Haney]

Attachment: ups the above was the wrong file

Comment 3

[Bernd]

thats a result of an optimization to avoid to assign space  to empty columns
(columns that have no cells originating) at the end of a table. If somebody
wants to fix that it requires some heavy digging in BasicTableLayoutStrategy.cpp
and will break older regression testcases (probably).

Comment 4

[Madhur Bhatia]

so.... what will be the fate for this bug ?

I do see the bug on winXP.

Comment 5

[Bernd]

I believe what we do is reasonable, we suppress these bogus colspans at the end.
If somebody would like to work on it would require to change the whole effective
column stuff. I couldnt find a reference how to handle these in my opinion
colspans at the end.

Comment 6

[Bernd]

Opera8.01 renders the testcase like we do for the first testcase, IE6 like we do
for the second testcase. As I said before this bug is somehow valid but really
low priority on my agenda.

Comment 7

[Dennis Haney]

Still not fixed :( This bug is really annoying when you autogenerate tables

Comment 8

[Bernd]

we render this identical to IE, probably fixed by bug 300030