167699 - Definition Lists: Indenting errors with the <dt> and <dd> tags, sometimes no break before/after <dd>...</dd>

Status: RESOLVED DUPLICATE of bug 117738

(Core :: DOM: HTML Parser, defect)

Description

[Joachim Jansen]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.2a) Gecko/20020910
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.2a) Gecko/20020910

Items after 'Products' - beginning with 'CompactPCI' up to 'Price Lists' should
be  indented and each displayed in a new line

Reproducible: Always

Steps to Reproduce:
1. Simply call http://www.ekf.de/products.html with Mozilla
2. Verify against IE
3.

Comment 1

[James Ross]

Confirming on Win2000, build 20020907 (1.2a).

I can see the problem clearly using the DOM Inspector (very useful tool) - the
DOM tree for that area of the document is as follows:

  +-P
  +-DL
  |  --DT
  |     --FONT
  |        +-FONT
  |        |-#text
  |        +-FONT
  |        |-#text
  +-#text

Clearly this isn't the intended structure - and I can't even see how Moz created
it. :)  I suspect this is actually to do with the site being rendered in "Quirks
mode". I'd therefor say it's really an evangelism issue (i.e. website's at
fault, not Mozilla), but it's still an impressivly messed up DOM tree.

Comment 2

[Henri Sivonen (:hsivonen)]

The site used invalid markup: garbage in, garbage out. It seems that the
"residual style" parsing somehow omits <dd>s when there is a <font> element in
the tree between <dl> and the <dd>s.

If this was to be worked around in Mozilla (not worth the trouble, IMO), this
would be a parser thing--not a layout issue. I suggest resolving as INVALID, but
sending over to parser for the decision.

Comment 3

[Richard Brodie]

Related: bug 117738, bug 146358.

Comment 4

[Boris Zbarsky [:bzbarsky]]

This is a duplicate of the various other "residual style confused in <dl> with
<dt> and <dd> tags that omit the end tags" bugs....

Comment 5

[Elmar Ludwig]

Marking as dup of bug 117738 (which is essentially the same issue).

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