22623 - An extra </li> displays wrongly as a <li> bullet <li>A <li>B <li>C</li> </li> </ul>

Status: VERIFIED FIXED

(Core :: Layout, defect, P3)

Description

[Daniel Weinstein]

Here is a code snippet that shows the bug:

  <ul>
	  <li>A
	  <li>B
	  <li>C</li>
	  </li>
  </ul>

Comment 1

[Daniel Weinstein]

Attachment: the code snippet with two consequtive </li> tags

Comment 2

[rickg]

Harish -- I'm giving this to you because you added the code that is responsible.
In HandleEndToken(), there's code in the default section of the switch() that
forces an open tag if only a close tag is found. I think you added it so
that we'd get the extra linefeed. I prefer that this code go away; perhaps we
should just emit a <BR>. Nonetheless, here you go.

Comment 3

[harishd]

FIXED by not opening an LI for stray /LI.

Comment 4

[Jim Dunn]

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

Comment 5

[Chris Petersen]

Fixed in the Jan 20 build.

Comment 6

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

SPAM. HTML Element component deprecated, changing component to Layout. See bug
88132 for details.