146274 - Huge indent on first half of web page, tables misplaced

Status: RESOLVED FIXED

(Core :: DOM: HTML Parser, defect, P4)

Description

[Blair Barnett]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0rc2) Gecko/20020510
BuildID:    

This web page looks fine in NS 4.77 (Linux), and IE 5.5 (Windows).  But the blue
text blocks at the top of the page get a huge indent, and the first question of
the Q and A paragraphs that should be halfway down the page gets pushed over to
the right of the blue text blocks.

Reproducible: Always
Steps to Reproduce:
1. Open http://www.hp.com/rnd/support/faqs/8000_4000_2424.htm
2.
3.

Actual Results:  As described above.

Expected Results:  Text blocks should be left-justified, and question paragraph
should be below links to Q and A section.

Comment 1

[R.K.Aa.]

seeing this on a day old cvs, linux, as well.
To the upper left of page, a single comment end-tag displays:

-->

Trying parser for component since there's been some comment horkage lately

Comment 2

[Heikki Toivonen (remove -bugzilla when emailing directly)]

This document has the following doctype:

"-/W3C/DTD HTML 4.0 Transitional/EN"

The one they wanted to use was

"-//W3C//DTD HTML 4.0 Transitional//EN"

This triggers strict mode as described in
http://www.mozilla.org/docs/web-developer/quirks/doctypes.html

Somewhere in the middle of the document they have this comment:

<!------------------------------------------   START ANSWERS TABLE  ---------------------------------------->	

which is not a good comment (every "--" needs to be closed by a matching "--"
and that does not happen above). So what happens is that our parser goes past
the end of that comment until the double dashes match and we see '>'; in other
words we eat a big chunk of the page into a comment. 

We could either evangelize, or what I would suggest instead, add the above (and
similarly wrong) doctypes to our quirks doctype list.

Comment 3

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

Adding that to the quirks list seems reasonable.

Comment 4

[harishd]

Attachment: patch v1.0

Added -/W3C/DTD HTML 4.0 Transitional/EN to the quirks list.

Comment 5

[harishd]

Fixed on 12/03/02