159108 - border-collapse: collapse. table borders should be reported as overflow

Status: RESOLVED FIXED

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

Description

[Martin Vollrathson]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.1b) Gecko/20020722
BuildID:    2002072204

On the test page, the border on the top and left sides of the table disappears.
It actually appears as if the border disappears halfway in under the
content/background och the cells. If the border is 4px wide, 2 pixels will be
visible at all sides of the table. If the border is 2px wide, 1 pixel will be
visible at all sides, etc. 1 pixel obviously becomes either 1 or 0 pixels.

Without border-collapse: collapse, everything works as expected.

Reproducible: Always
Steps to Reproduce:
1. Load the test page.
2. Look at the borders around the table.

Actual Results:  The border is invisible (or width: < 1px) on the top and left
sides.

Expected Results:  There should be 1px borders on all sides.

Comment 1

[Amarendra Hanumanula]

I could see this problem only on the trunk builds but not on brach builds. So 
some checkins into the trunk might have caused this problem. Works fine on 
2002-07-29-08 branch build. But the problem appears on 2002-07-29-08 trunk 
builds.

Comment 2

[Michal Kubecek]

Attachment: another testcase

Another testcase. If 'position: absolute' is changed to 'position: fixed', the
border is rendered on all sides.

Comment 3

[Andrew Schultz]

regression bewteen linux trunk builds 2002061108 and 2002061304
I don't see any relevant checkins during that window.

Comment 4

[Bernd]

Attachment: testcase

testcase demonstrating that the outer half of the border is not drawn

Comment 5

[Bernd]

The problem here is that tables with collapsed borders have the half of its
outer border beyond the table region itself
(http://www.w3.org/TR/REC-CSS2/tables.html#collapsing-borders). They should
report those leaking content in the metrics.mOverflowArea, but they dont.

Comment 6

[Bernd]

A quick and easy way to solve this problem would be to make the quirks rendering
our standards rendering. We would then have two interoperable implementations
(IE) and could go to the w3c to remove the margin area from CSS2.1.

Comment 7

[Hixie (not reading bugmail)]

Someone would have to propose that to the WG of course.

Comment 8

[Bernd]

I proposed it at http://lists.w3.org/Archives/Public/www-style/2003Jan/0230.html

Comment 9

[Jan Kunzmann]

Attachment: Workaround

A workaround for this problem is to add an 1 pixel margin to the left/top of
the table as shown in the testcase.

Comment 10

[Max Kanat-Alexander]

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

Comment 11

[Max Kanat-Alexander]

The first two testcases and the URL WFM on Moz 1.4final Win2K.

The demonstration of the error in "Workaround" still shows the error.

This seems to suggest that there are actually two bugs here, and the first one
is WFM. It seems like the second one is described well in comment 5.

Also, did the WG ever have anything to say about the proposal?

-M

Comment 12

[Bernd]

the table border will be reported as overflow once I get the patch in bug 173277
reviewed. fantasai puts the groundwork to have a margin around bc tables in  bug
4510.

Comment 13

[José Jeria]

Attachment: Screenshot of weird error when viewing testcase "Wordaround"

Dont know if this should be a new bug, but when viewing the testcase named
"Workaround":
1. Take any window and drag it above the Mozilla windows. 
2. Drag away the windows from Mozilla and you will see another border that
wasnt there (see screenshot)

Using Gecko/20030914 Firebird/0.6.1+, Windows XP

Comment 14

[Mats Palmgren (inactive)]

URL and testcases works fine now that bug 173277 is fixed.
Bug 155955 covers the border-collapse border/origin/margin discussion.

FIXED by bug 173277