5799 - td width=% problems when table width is calculated

Status: VERIFIED FIXED

(Core :: Layout: Tables, defect, P3)

Description

[Péter Bajusz]

The following two tables should look the same,
they should have two identical width cells.
None of the above holds for gecko (1999042908).
As always, both ie4 and ns4.5 are ok.
---------
<table border=5>
  <tr>
    <td width=50%>almaalmaalmaalmaalma</td><td>x</td>
</table>
<p>
<table border=5>
  <tr>
    <td>almaalmaalmaalmaalma</td><td width=50%>x</td>
</table>

Comment 1

[karnaze (gone)]

Moving to M9.

Comment 2

[Péter Bajusz]

The testcase is not about having TWO tables together,
so I'm making them two separate testcases.
Adding as attachments for easier viewing from the browser.

Comment 3

[Péter Bajusz]

Attachment: Testcase #1 from the description

Comment 4

[Péter Bajusz]

Attachment: Testcase #2 from the description

Comment 5

[Péter Bajusz]

With 1999-07-31-08 build testcase #1 looks better, but testcase #2 is still bad.
I see a litle extra space in the left cell of testcase #1, but I can only see
that in Apprunner, not in Viewer! This is very strange...

Comment 6

[Péter Bajusz]

*** Bug 13484 has been marked as a duplicate of this bug. ***

Comment 7

[Péter Bajusz]

Testcase from bug 13484 is
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=1626

Comment 8

[leger]

Spoke to karnaze during bug triage today.  Moving to M11.  Not an M10 blocker.

Comment 9

[karnaze (gone)]

Adding keywords - autoTable, percentCell

Comment 10

[karnaze (gone)]

Fixed with latest checkin.

Comment 11

[Péter Bajusz]

My #2 testcases is working now, but I just found this testcase
an incomplete one.
In this case the cell asks for more space, so its percentage can be higher.

(It's the opposite case than in #1 when the cell asks the table to grow,
so it's percentage can be _lower_)

While it works if only one cell asks for more space,
when there's more than one it does not grow the table enough.

Try:
<table border=5><tr>
    <td>some text</td><td width=40%>x</td><td width=40%>x</td>
</tr></table>
---
The table should grow more as the percentCells fight for space.

Also:
<table border=5><tr>
    <td>some text</td><td width=50%>x</td><td width=50%>x</td>
</tr></table>
---
In this case the desired with cannot be justified, and in Nav4 (and IE4)
the table grows as large as it can.
This is also why the testcase from bug 13484 is still broken.

I'm reopening the bug.

Comment 12

[Péter Bajusz]

Attachment: Testcase for two % cell fighting for more space

Comment 13

[karnaze (gone)]

Fixed with 10/24 pm checkins.

Comment 14

[Péter Bajusz]

Testing with 1999102516 mozilla.exe and viewer.exe (std mode) on Win98.
All testcases work well including the one from the duplicate bug.
I'm marking the bug VERIFIED.