Line break converter (in filter.module) does not handle contents of li elements (lists) correctly. The result is invalid html, due to broken p elements.

<ol>
<li>Ok, so isn't this an issue with line break converter anyway?</li>
<li>What if I have just an ordinary list

with line breaks in between?</li>
<li>It's not gonna work then either
is it?</li>
</ol>

Produces the following:

<ol>
<li>Ok, so isn't this an issue with line break converter anyway?</li>
<li>What if I have this kind of list
<p>with line breaks in between?</li>
<li>It's not gonna work then either<br />

is it?</li>
</ol>

This example from currently running drupal.org, as you can see:

  1. Ok, so isn't this an issue with line break converter anyway?
  2. What if I have just an ordinary list

    with line breaks in between?

  3. It's not gonna work then either
    is it?

Comments

magico’s picture

magico commented
Version: 4.7.0 » x.y.z

Still present.

beginner’s picture

beginner commented
Version: x.y.z » 5.x-dev

subscribing.

beginner’s picture

beginner commented
Version: 5.x-dev » 6.x-dev
lilou’s picture

lilou commented
Status: Fixed » Closed (fixed)

This bug seem to be solved in D6-RC1.

Test produce :

  1. Ok, so isn't this an issue with line break converter anyway?
  2. What if I have just an ordinary list

    with line breaks in between?

  3. It's not gonna work then either

    is it?
beginner’s picture

beginner commented
Status: Active » Fixed
Anonymous’s picture

Anonymous (not verified) commented

Automatically closed -- issue fixed for two weeks with no activity.