3000 - collapsing border model table margins

Status: VERIFIED FIXED

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

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

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

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Whoops... wrong second URL.  That should have been see the diagram in
http://www.w3.org/TR/REC-CSS2/tables.html#collapsing-borders

Comment 2

[Paul MacQuiddy]

per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser

Comment 3

[karnaze (gone)]

Moving to M9.

Comment 4

[Erin Pierce]

Attachment: test case for 3000

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

[e-mail] : your testcase is actually a text description of another
bug.  It looks like you attached the wrong file.

Comment 6

[karnaze (gone)]

Adding [CSS 2] to summary.

Comment 7

[karnaze (gone)]

mass move to m14.

Comment 8

[Hixie (not reading bugmail)]

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...

Comment 9

[karnaze (gone)]

Moving to M16.

Comment 10

[karnaze (gone)]

Moving to M17.

Comment 11

[karnaze (gone)]

*** This bug has been marked as a duplicate of 41262 ***

Comment 12

[karnaze (gone)]

I should have made a dependency instead of a dup. 

Comment 13

[rbs]

Attachment: Another test case

Comment 14

[rbs]

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.

Comment 15

[Oliver Klee]

Milestone 0.8 has been released. We should either resolve this bug or update its
milestone.

Comment 16

[karnaze (gone)]

Moving to m1.0

Comment 17

[Amarendra Hanumanula]

QA contact update

Comment 18

[anthonyd]

this looks fixed on my new build.
anthonyd

Comment 19

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

That's because our support for the collapsing border model was turned off (but
not removed, I don't think).

Comment 20

[karnaze (gone)]

collapsing border bugs moved to m098

Comment 21

[karnaze (gone)]

->m099

Comment 22

[karnaze (gone)]

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. 

Comment 23

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

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.

Comment 24

[karnaze (gone)]

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".

Comment 25

[karnaze (gone)]

I forgot to add that IE and Opera both appear to implement the collapsing border 
model, although not very well.

Comment 26

[karnaze (gone)]

Attachment: test case

Comment 27

[Kevin McCluskey (gone)]

Marking nsbeta1+

Comment 28

[karnaze (gone)]

Fixed by meta bug 41262.

Comment 29

[Amarendra Hanumanula]

 Works fine with the build 2002030403. Marking verified