Closed Bug 2038 Opened 26 years ago Closed 25 years ago

UL, etc. left margin vs. padding

Categories

(Core :: CSS Parsing and Computation, defect, P2)

Other
Other
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: dbaron, Assigned: peterl-retired)

References

Details

Attachments

(1 file)

I noticed in ua.css you have 40px of margin-left on UL, DIR, MENU, and OL. I think it would be better to make this padding, because of drawing background- colors or borders on ULs and placing them next to paragraphs. If it is margin, then it would be unknown on which side the bullets would fall - it would depend on the LI box properties. I think padding makes more sense, even though it is not what is suggested in http://www.w3.org/TR/REC-CSS2/sample.html .
Perhaps the list indentation should be on the LI's margin rather than the UL's padding. That would solve the inside/outside bug as well. However, I think UL padding or LI margin are the only two good alternatives. I think UL margin is bad. Then a background wouldn't get the bullets... and borders would be messy.
I think this is related to bug 1363.
Setting all current Open/Normal to M4.
Status: NEW → ASSIGNED
Target Milestone: M4 → M7
*** Bug 1363 has been marked as a duplicate of this bug. ***
Attached file testcase showing problems (deleted) —
You may also want to have this padding (or margin, if you insist) in em units rather than px.
Target Milestone: M10 → M11
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Changed lists to use padding for indents. This isn't quite backward compatible, but it's better than what we had before (and probably better for the authors this way).
Status: RESOLVED → VERIFIED
Using 9/16 Apprunner with attached test case, verified fixed. Padding now used for indents and page displays with list indicators inside box.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: