1378 - {vmargin} collapsing margin problems

Status: VERIFIED FIXED

(Core :: Layout, defect, P2)

Description

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

The top and bottom margins of the ULs in the above page seem to disappear, but
they are actually there (I think), but just not colored with the background
color of the DIV that surrounds them.

Comment 1

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

I just realized this is a collapsing margins problem, but I still think your
current choice of colors in the collapsing margin is not optimal, especially in
the middle one where both sides are the same color (there is a </DIV><DIV>
between the margins, but that shouldn't make a difference).

Comment 2

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

At the URL http://www.w3.org/Style/CSS/Test/current/sec411.htm , nested margins
do not collapse.  However, you are collapsing them in the URL above.  They
should be collapsed, per the spec, but I don't like it, because of what happens
in the URL above (http://www.fas.harvard.edu/~dbaron/csstest/listbox.html).

Comment 3

[leger]

Setting all current Open/Normal to M4.

Comment 4

[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 5

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

Actually, based on the errata for section 10.6.3, you are currently handling
this correctly:

http://www.w3.org/Style/css2-updates/REC-CSS2-19980512-errata.html

That part of the bug is invalid.

However, the problems with classes five/six/seven of
http://www.w3.org/Style/CSS/Test/current/sec411.htm remain.

Comment 6

[kipp]

The test lays out correctly. note that the bullets for the first div section's
3rd div (the "condensed" one) will be invisible because they are off the screen
to the left...

Comment 7

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

Verified fixed.