Closed Bug 3000 Opened 26 years ago Closed 23 years ago

collapsing border model table margins

Categories

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

x86
Windows 95
defect

Tracking

()

VERIFIED FIXED
mozilla0.9.9

People

(Reporter: dbaron, Assigned: karnaze)

References

()

Details

(Keywords: css2, testcase, Whiteboard: [awd:tbl])

Attachments

(1 file, 2 obsolete files)

Collapsing border model table margins aren't quite done correctly. The outer half of the border should be in the margin. Thus the left edges of the first and second tables in the above URL shouldn't line up. See the diagram in http://www.fas.harvard.edu/~dbaron/csstest/tborder1.html
Whoops... wrong second URL. That should have been see the diagram in http://www.w3.org/TR/REC-CSS2/tables.html#collapsing-borders
Status: NEW → ASSIGNED
per leger, assigning QA contacts to all open bugs without QA contacts according to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
Target Milestone: M7
Moving to M9.
Attached file test case for 3000 (obsolete) (deleted) —
erin@imaginet.com : your testcase is actually a text description of another bug. It looks like you attached the wrong file.
Summary: collapsing border model table margins → [CSS 2] collapsing border model table margins
Adding [CSS 2] to summary.
Summary: [CSS 2] collapsing border model table margins → {css2} collapsing border model table margins
mass move to m14.
Keywords: css2
Migrating from {css2} to css2 keyword. The {css1}, {css2}, {css3} and {css-moz} radars should now be considered deprecated in favour of keywords. I am *really* sorry about the spam...
Moving to M16.
Target Milestone: M14 → M16
Summary: {css2} collapsing border model table margins → collapsing border model table margins
Moving to M17.
Target Milestone: M16 → M17
*** This bug has been marked as a duplicate of 41262 ***
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → DUPLICATE
I should have made a dependency instead of a dup.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Status: REOPENED → ASSIGNED
Depends on: 41262
Attached file Another test case (obsolete) (deleted) —
I just attached a test case which demonstrates that 'margin-top' is not handled properly. I guess the rework of the collapsing/margin code should fix this problem as well.
Keywords: testcase
Milestone 0.8 has been released. We should either resolve this bug or update its milestone.
Target Milestone: M17 → ---
Moving to m1.0
Target Milestone: --- → mozilla1.0
QA contact update
QA Contact: chrisd → amar
Whiteboard: 5/21: reporter contacted for additional information
Whiteboard: 5/21: reporter contacted for additional information
this looks fixed on my new build. anthonyd
Whiteboard: [awd:tbl]
That's because our support for the collapsing border model was turned off (but not removed, I don't think).
Target Milestone: mozilla1.0 → mozilla0.9.7
collapsing border bugs moved to m098
Target Milestone: mozilla0.9.7 → mozilla0.9.8
->m099
Target Milestone: mozilla0.9.8 → mozilla0.9.9
I think in quirks mode, the outer half of the table border should not be in the table margin. This will be compatible with IE (hence the quirk) and will not result in clipped borders if a table margin is not big enough. I'm not sure why the spec is the way it is, but it seems like it is going to be a pain for authors to have to figure out how big to make a table margin.
Other browsers don't implement the collapsing border model, so there's no reason to deviate from the spec. We should only have quirks when there are significant numbers of web pages that depend on an incorrect behavior. If you really think the spec is wrong, you should raise the issue with the CSS working group and try to get the spec changed.
Well, before trying to get the spec changed, I would like to amend the criteria for quirks mode to include "if all or most browsers implement a particular item contrary to the spec and it seems natural (at least to the user agent developers) to do it that way".
I forgot to add that IE and Opera both appear to implement the collapsing border model, although not very well.
Attached file test case (deleted) —
Attachment #831 - Attachment is obsolete: true
Attachment #11135 - Attachment is obsolete: true
Marking nsbeta1+
Keywords: nsbeta1+
Fixed by meta bug 41262.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago23 years ago
Resolution: --- → FIXED
Works fine with the build 2002030403. Marking verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: