161562 - misplaced bullet in HTML list

Status: RESOLVED DUPLICATE of bug 100930

(Core :: DOM: HTML Parser, defect)

Description

[Andrew Saturn]

as see at this URL:

http://www.palacespace.com/downloads.php

where there is a <ul>, and then descriptive text, and *then* a <li>, the first
list item bullet is shown where you'd expect the first list item to be ... and
it is then corrected with the second list item.

using build 2002071308
updating to a more recent build ... will reply with the results

Comment 1

[Andrew Saturn]

confirmed on "nightly" build 2002080713

Comment 2

[Kai Lahmann (is there, where MNG is)]

well, such a text isn't valid, but it still shouldn't do that...

Comment 3

[Johnny Stenback (:jst)]

Not a DOM problem, over to layout.

Comment 4

[alge]

The linked URL doesn't show this (there is no <ul> [descriptive text] <li>)

Comment 5

[Boris Zbarsky [:bzbarsky]]

Attachment: testcase

I see the "descriptive text" indented, but all the bullets are in the right
places...

do you have a testcase that actually shows a problem here? ;)

Comment 6

[Andrew Saturn]

the site is a production site, and so, the page was fixed by another webmaster,
and the broken copy was deleted. sorry, but it's out of my hands... I'm only the
designer for it: the content guys do the text part (which is where I noticed the
bug)

leaving this as unconfirmed until someone else notices it.

Comment 7

[Boris Zbarsky [:bzbarsky]]

Well... any way of getting a copy of the original "broken" markup?  Revision
history for the site, maybe?

Comment 8

[Andrew Saturn]

the "old copy" randomly stopped showing the bug ...

oldest one I have on my own computer:
http://www.palacespace.com/old_downloads.php

I *originally* saw:

--
  Title
*
  item1
* item2
* item3
--

... but the page appears to be showing it correctly now, while using a non <li>
as the first item after the <hr>. I looked for a copy that would show it, but
nothing did.

I know for a fact it appeared on this build (2002080713), I just don't get it ...

if anyone finds it again, they'll be sure to find this report (if they search
bugs correctly), and I'll continue to try to get it to happen again.

Comment 9

[Andrew Saturn]

as my grandpa used to say, "if it was a snake, it woulda bit ya"

I found the bugged file, uploaded here:
http://www.palacespace.com/mozillabug_downloads.html

shows the bullet bug on build 2002080713


though the code isn't perfect, Mozilla still shouldn't do that. IE5.5 for
Windows shows the code as intended by the original author.

Comment 10

[Boris Zbarsky [:bzbarsky]]

Attachment: Minimal testcase

OK.  There we go...

Comment 11

[Boris Zbarsky [:bzbarsky]]

So to get this to happen you must have code that looks like this:

<a>
<li> Some text
<ul><a> Some more text </a></li>
</ul>

The problem here is that the outer <a> needs to span the <li> but cannot span
the inner <a>, so it needs to be closed before we get there....  The content
model generated by the parser in this case is:

A (whitespace-only text node inside)
UL
|\_A
|  |\_Some text
|  |
|   \_LI (empty)
 \_A
    \_Some more text

Given that content model, layout is actually correct an all. ;)

Over to the parser.

Comment 12

[Andrew Saturn]

changed name and discription, added keywords

I believe it has to do with <a> tags, because when I removed those when trying
to test it, it appeared fine.

Comment 13

[Mats Palmgren (inactive)]

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