Applying the div tool from the toolbar to a list screws the list up

[Exception e]

Applying the div tool from the toolbar to a list screws it up.

Type and select:

1. foo
   
2. bar
   
3. roo
   

Then apply the div tool. Give it an inline style "border: 1px solid blue;".

Result:

paragraphs are added with contents of the li items. The list contains only spaces. The paragraphs are wrapped in the new div, but the list is outside the div.

---

tested both ie7 and firefox3

[Martin Kou]

Confirmed.

The bug also happens with the blockquote command.

[Frederico Caldeira Knabben]

The changes in the patch are too risky for a beta stabilization. I'm moving it to the next release.

[Exception e]

But the buggy div tool is quite useless without this fix. Now blockquotes and divs are unreliable. Maybe it's an idea then to publish 2.6.4beta simultaneously.

[Martin Kou]

If you need the fix urgently, you can download the source code of 2.6.3 from ​https://svn.fckeditor.net/FCKeditor/tags/2.6.3/ with an SVN client, apply the patch, and compile the code package with the _dev/build_release.bat file. You'll need a Windows machine with PHP installed to compile the package.

[Alfonso Martínez de Lizarrondo]

This patch breaks the packaged files.

to fix it I've added a semicolon at the end of the noBlockLeft function:

			var noBlockLeft = function( bqBlock )
			{
				for ( var i = 0 ; i < bqBlock.childNodes.length ; i++ )
				{
					if ( FCKListsLib.BlockElements[ bqBlock.childNodes[i].nodeName.toLowerCase() ] )
						return false ;
				}
				return true ;
			} ;

[Martin Kou]

Ok, updated patch with the semicolon added.

[Martin Kou]

Fixed with [2598].

Click here for more info about our SVN system.