Bug 100279 – Wrong column widths when a row has colspan and long unwrappable text and other rows cells width is specified with percentage

• Component: Tables
• Status: ASSIGNED

[Nicolás Cardelino]

Created attachment 170446 [details]
Last row cell with colspan=2

If the last row of a table has colspan > 1 and contains a long unwrapped text, and the width of the first cell of previous rows is specified using percentage (for example 10%), those previous cells ignore the width and take 50% instead.

Given a 3 row table, with 2 columns where the last row has colspan = 2, and the first cell of the first 2 rows width is 10%, instead of this:

 10%
*--*------------------*
*--*------------------*
*---------------------*

you get this:

*----------*----------*
*----------*----------*
*---------------------*

[Nicolás Cardelino]

Created attachment 170449 [details]
Last row cell with colspan=2

[alan]

Created attachment 200647 [details]
nowrap

Both Opera and Firefox disagree with WebKit rendering.

[alan]

<tr><td style="width:90%;"></td><td></td></tr>
<tr><td colspan="2"></td></tr>

in AutoTableLayout::calcEffectiveLogicalWidth(), we ignore the percent value, when the min/max column widths are calculated (colspan is present). The second column's auto width value negates the percentage value so the available width distribution is based on the cell's content width instead.
Shouldn't we do something like

if (percent and auto columns only)
    distribute (100-percent) across the auto columns

and go ahead with the 'allColsArePercent' calculation?

[Ahmad Saleem]

Created attachment 477910 [details]
Safari Technology Preview vs Chrome vs ToT

[Ahmad Saleem]

It is now fixed partially but not fully because if we see `text` in second row is not matching other, so keeping it open.