DL-inside-UL-inside-DL loses indentation

VERIFIED FIXED in M17

Status

()

defect
P3
normal
VERIFIED FIXED
20 years ago
19 years ago

People

(Reporter: newt, Assigned: harishd)

Tracking

({regression})

Trunk
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

20 years ago
The URL above (or http://www.cdrom.com/pub/png/) shows the problem pretty
obviously:  the innermost DLs, used for the "...with a background image" lines,
lose all indentation and end up at the left margin.  A similar test page without
the outermost DL (http://pobox.com/~newt/png-moz-test2.html) renders with
correct indentation, but it includes extra vertical space around the DL that is
not present in Navigator 4.x.  I don't know if the latter issue is more or less
correct in Mozilla, but I personally prefer to add my own vertical space if I
need it (i.e., the NN 4.x behavior is better).

Updated

20 years ago
Assignee: troy → rickg
Component: Layout → Parser

Updated

20 years ago
Status: NEW → ASSIGNED
Target Milestone: M6

Comment 1

20 years ago
Corrected by changes to parsing engine.

Updated

20 years ago
Status: RESOLVED → VERIFIED

Comment 2

20 years ago
Fixed in 5/17 Linux Build
(Reporter)

Comment 3

19 years ago
Linux build from sourceball (2000-04-14, ~22:30 PDT) once again demonstrates the
old behavior from a year ago.  Two of the test URLs have moved:

	http://pobox.com/~newt/test/png-moz-test.html
	http://pobox.com/~newt/test/png-moz-test2.html
	http://www.cdrom.com/pub/png/

Greg Roelofs
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Whiteboard: broken again
(Reporter)

Comment 4

19 years ago
adding regression and 4xp keywords
Keywords: 4xp, regression

Comment 5

19 years ago
How come this is still M6? M6 is out for ages!

Comment 6

19 years ago
It was originally opened in m6, and has regressed. We're not quite *that* lame.

Comment 7

19 years ago
Still so, leaving the m6 mark is meaningless, 
I think that milestone should represent the current, not the original 
schedule.
(Assignee)

Comment 8

19 years ago
Reassigning to myself. And setting to M17.
Assignee: rickg → harishd
Status: REOPENED → NEW
Target Milestone: M6 → M17
(Assignee)

Comment 9

19 years ago
Do you still see the problem? B'cos I'm not seeing it.
Status: NEW → ASSIGNED
(Reporter)

Comment 10

19 years ago
No, it appears to be fixed--thank you to whomever was responsible.  I checked
with the 2000-05-28-09-M16 Linux binary, and all indentation and vertical
spacing appears identical to NN 4.x.  Whiteboard cleared, resolution changed to
FIXED.

Greg
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
Whiteboard: broken again

Comment 11

19 years ago
Marked verified fixed in the May 31 build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.