Extra block added before Bulleted list

[PavelZ]

Steps to reproduce:

In fck do: aaaaaaa <Shift+Enter> bbbbbbbbbbb <Shift+Enter> cccccccc

on the second line (with "bbbbbbb") press "bulleted list". Second line is bulleted but an empty line appears between line 1 and 2 (in IE always in FF sometimes).

The Source HTML is:

<p>aaaaaaa<br /> &nbsp;</p> <ul>

<li>bbbbbbbbbbb</li>

</ul> <p>cccccccc</p>

Tested on FCK demo (ver. 2.5 Beta Build 16848)

[fredck]

Confirmed with IE. Works well with FF.

[martinkou]

It seems the problem originates from FCKDomRangeIterator, instead of the list command logic.

After some experiments, I found FCKDomRangeIterator isn't deleting <br> tags at the end of splitted block elements correctly. For example, say we have this:

<p>
aaaaaa<br />
b|bbbbb<br />
cccccc
</p>

With the | character denoting the caret position.

Now let's run the following code:

r = new FCKDomRange(FCK.EditorWindow);
r.MoveToSelection();
i = new FCKDomRangeIterator(r);
i.ForceBrBreak = true; // this flag is used in list creation
i.GetNextParagraph();

The expected behavior of the above code block is that middle line gets split out into an independent paragraph:

<p>aaaaaa</p>
<p>bbbbbb</p>
<p>cccccc</p>

But what happened with the current code is this:

<p>aaaaaa<br /></p>
<p>bbbbbb</p>
<p>cccccc</p>

There's an extra <br /> in the first paragraph which shouldn't be there.

[martinkou]

Fixed with [1090].

Click here for more info about our SVN system.