Closed Bug 3000 Opened 22 years ago Closed 19 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)

743 bytes, text/html
Details
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) —
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: 20 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) —
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
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: 20 years ago19 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.