59252 - fixed layout tables overflow its content

Status: VERIFIED FIXED

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

Description

[Henrik Gemal]

The table at:
http://browserwatch.internet.com/news/news-current.html
becomes very wide.

Comment 1

[Henrik Gemal]

Attachment: testcase with troubled table

Comment 2

[Boris Zbarsky [:bzbarsky]]

I see this with linux trunk build 2000110521.

OS -> ALL

Comment 3

[James Lariviere]

Something has been changed over the past weekend on the table code because it
seems that nesting tables has some very wierd and wrong rendering problems.

I just confirmed bug 59278 and it seems like nested tables is the culprit in it too.

Look at this page (http://www.dglconsultants.com/sample.html) on a site I
manage.  Whatever has changed over the past few days completely mangles this
page too.

Comment 4

[James Lariviere]

Sorry for the SPAM.

What the hell happened to the build id in the title bar?

Mozilla/5.0 (Windows; U; Win98; en-US; m18) Gecko/20001106

Comment 5

[James Lariviere]

Adding keywords.

This is a very visible regression and is completely BREAKING correctly written html.

I downloaded a few dailies to track down when this regression happened.
Something caused this regression between the 10-28-08 trunk build and the 10-29
trunks (viewed in win98).

I queried bugzilla (how do you query for a specific day for bugs that are
fixed?) for the last 15 days for layout and html tables. I did not really find
anything that suggests what caused this.

WTF?

Another site that i noticed this bug is here:
http://www.annuitiesnet.com/v2/news.shtml

I will provide a screenshot.

Comment 6

[James Lariviere]

Attachment: bad rendering from oct 29th trunk build & bad as todays 11-08 trunk

Comment 7

[James Lariviere]

Attachment: fine rendering from 10-28 build

Comment 8

[James Lariviere]

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

Comment 9

[James Lariviere]

some comments made by John Morrison from bug 59278...

The core issue is that for
a nested table which is also 'table-layout:fixed;', the text contents of the
inner tables cells will have unconstrained width (and extend outside the
containing tables right border). This landed with limbo1 on Oct30th with
buster's checkin for bug 53690 [i.e., this bug is in the candidate builds].
This is bad news, as it means the use of 'table-layout:fixed' is a hazardous
exercise.

Comment 10

[karnaze (gone)]

Attachment: derived test with an additonal cell and more cellpadding

Comment 11

[karnaze (gone)]

The 1st patch to bug 53690 probably caused this bug. I have a patch which I'll 
attach but I want to look at http://www.dglconsultants.com/sample.html first.

Comment 12

[karnaze (gone)]

Attachment: patch to fix the bug

Comment 13

[James Lariviere]

Thanks for patching so quickly!

Any chance to get this checked in for trunk and branch?

Also, does your patch also fix the fixed layout table border problem reported in
bug 43854?

Thanks again.

Comment 14

[James Lariviere]

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

Comment 15

[James Lariviere]

Revising summary so you can find this bug easier and minimize dupes.

Comment 16

[karnaze (gone)]

The patch is checked in.

Comment 17

[James Lariviere]

verified. :-)

Will this get patched into the first major post-RTM point release of NS6?