128060 - Margins do not work on OL elements

Status: RESOLVED DUPLICATE of bug 27851

(Core :: CSS Parsing and Computation, defect)

Description

[Georg Maaß]

Mozilla ignores margin settings on positioned OL elements

There might be a relation to http://bugzilla.mozilla.org/show_bug.cgi?id=122883

Comment 1

[Georg Maaß]

Attachment: zipped test case

Comment 2

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

A simpler testcaes would be nice.  You'd also better clearly describe what you
expect to happen in the testcase since I can't see what's wrong.

Comment 3

[Georg Maaß]

Attachment: zipped test case

The previous attachement included the wrong stylesheet. Now it includes the
correct one.

Comment 4

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

I still don't see the problem (even after correcting a bunch of relative URLs
that assumed the ZIP would be unzipped in the root directory).

Could you:
 * attach a testcase to the bug that can be clicked on from the bug (a single
HTML file with CSS and JS inside of it), as a text/html file rather than
application/x-zip-compressed
 * describe what you expect to see in that testcase and how that is different
from what actually happens.

Comment 5

[Georg Maaß]

OK, I try to build a all in one test case.

Comment 6

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

I'm wondering if the "problem" you're seeing is the result of bug 2038, though.
 Do you really want to specify 'padding' and 'margin' rather than just
specifying 'margin' and leaving 'padding' as the UA default (which can vary
depending on how the default indentation is implemented).

Comment 7

[Georg Maaß]

Attachment: Now a all in one shortened test case

Comment 8

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

You still haven't said what it is that you expect to happen and how Mozilla's
behavior differs from that.

Comment 9

[Georg Maaß]

I want 1 em indention, this works well with IE by setting the margin but is
ignored by Mozilla and my writings are also ignored by multiple MID-AIR-COLLISIONs

Comment 10

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

You need to add 'padding-left: 0' where you have 'margin-left: 1em'.  (You
should probably also specify both 'margin-left: 0' and 'padding-left: 0' for the
LI elements.)  The CSS spec doesn't say whether the default indendation on lists
should be treated as margin or padding (or whether it's on UL or LI), and we
have chosen the more sensible option, which is padding.  See bug 2038 and
http://www.people.fas.harvard.edu/~dbaron/css/user/boxmodel .

Comment 11

[Georg Maaß]

Ok, I'll try this tomorrow, when I'm back to my working place. If trhis works, I
build a third layout stylesheet which contains the differences between Mozilla
and IE.

Comment 12

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

Putting in 'padding-left: 0' isn't going to change anything in IE.  You don't
need to sniff stylesheets.

Comment 13

[Georg Maaß]

OK, using the padding results in the desired look. Thanks for advicing me.

Comment 14

[Hixie (not reading bugmail)]

VERIFIED

Comment 15

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

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

Comment 16

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

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

Comment 17

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

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