484256 - [BC]Table with "border-collapse: collapse" & huge border overlaps nearby content

Status: RESOLVED FIXED

(Core :: Layout: Tables, defect)

Description

[Daniel Holbert [:dholbert]]

Attachment: testcase 1

STR: Load attached testcase.

ACTUAL RESULTS: "Text after table" is placed too high -- it overlaps the table border.

EXPECTED RESULTS: "Text after table" should be placed *after* the table border.

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2a1pre) Gecko/20090318 Minefield/3.6a1pre

Opera v9.64 shows expected results.  I also get expected results if I remove the "border-collapse: collapse" style.

Comment 1

[Daniel Holbert [:dholbert]]

Attachment: testcase 2

Here's another broken testcase, using border-top & text *above* the table.  In this case, we place the table too high up, so its border overlaps the text (and actually extends off the top of the page).

Comment 2

[Daniel Holbert [:dholbert]]

Firefox 3.0.x and 2.0.0.x have the same (broken) behavior on these testcases as mozilla-central.  So, not a regression.

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.7) Gecko/2009030422 Ubuntu/8.10 (intrepid) Firefox/3.0.7
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.18) Gecko/20081029 Firefox/2.0.0.18

Comment 3

[Bernd]

this is due to limiting the border width to 255 px, dbaron removed a lot but not all occurrences.

Comment 4

[Bernd]

Attachment: this fixes the bug

but lacks reftests

Comment 5

[Bernd]

Attachment: patch

Comment 6

[Bernd]

Attachment: patch

Comment 7

[Bernd]

http://hg.mozilla.org/mozilla-central/rev/2a3a2af10feb