169749 - Border affects paragraph margin in list item

Status: RESOLVED FIXED

(Core :: Layout: Block and Inline, defect)

Description

[Matt McClanahan]

In paragraph-list.html, the last list item's bullet is rendered above where it
should be.  I'm able to fix it by setting the margin-top of the first paragraph
in the list item to 0px.  However this only works if the list item has a border
on its top edge (the second list item).

This is only an issue for the list items which have <a name>'s in them (The
second and fourth).

Testing with Mozilla 1.0 (via Galeon) and 1.2a.

Comment 1

[Mats Palmgren (inactive)]

Attachment: More testcases

Comment 2

[Mats Palmgren (inactive)]

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

Comment 3

[Boris Zbarsky [:bzbarsky]]

Is this still a problem?

Comment 4

[Mats Palmgren (inactive)]

It's better, but still not according to spec AFAICT:
http://www.w3.org/TR/css3-lists/#markers
"Thus, markers are aligned with the first line of an element's content"

Comment 5

[Mats Palmgren (inactive)]

Attachment: Testcase #2, bullet should align with "P" in all cases

Comment 6

[Boris Zbarsky [:bzbarsky]]

.

Comment 7

[Lars Frers]

A real life example for this can be seen in the right coloumn of this page
http://userpage.fu-berlin.de/~frers/cgi-bin/blosxom.cgi
This bug appears with current trunk builds of Firefox and Camino too.

Comment 8

[Lars Frers]

With the checkin of bug 300030 (Refactor intrinsic width computation out of nsIFrame::Reflow) this bug seems to be resolved. :) Yay and thanks!

Comment 9

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

No, attachment 122225 [details] is still broken (the issue there is the vertical position of the bullets in the right column).

Comment 10

[Wayne Mery (:wsmwk)]

(In reply to comment #5)
> Created an attachment (id=122225) [details]
> Testcase #2, bullet should align with "P" in all cases

WFM, if I understand this correctly
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a3pre) Gecko/20070227 Minefield/3.0a3pre 

Comment 11

[Boris Zbarsky [:bzbarsky]]

Attachment: Regression tests

Comment 12

[Boris Zbarsky [:bzbarsky]]

Fixed by checkin for bug 367332.

Comment 13

[Boris Zbarsky [:bzbarsky]]

Added the tests.