Unexpected behavior when toggling bulleted/numbered list with a floating element

[Pablo Vanwoerkom]

Enter the following in the ckeditor.

<p>
	paragraph 1 with float left image <img src="http://a.cksource.com/c/1/inc/img/logo-ckfinder.gif" style="float: left;" /></p>
<p>
	paragraph 2</p>
<p>
	paragraph 3</p>

1. Select all text and then hit the bulleted/numbered list button. This will move the image into it's own bullet, separated from paragraph
2. If you click on a list button again, the image is now in it's own paragraph.
3. If you click it again, the image will disappear and its atrributes will get moved to the <li> tag and the bullet will just contain a &nbsp;.

This was tested in Firefox 3.5.

[Krzysztof Studnik]

Confirmed in CKE3.4.1, Firefox 3.6.10 result of above operations:

<ul>
	<li>
		paragraph 1 with float left image</li>
	<li src="http://a.cksource.com/c/1/inc/img/logo-ckfinder.gif" style="float: left;">
		&nbsp;</li>
	<li>
		&nbsp;</li>
	<li>
		paragraph 2</li>
	<li>
		paragraph 3</li>
</ul>

[Krzysztof Studnik]

linked with #6429

[Garry Yao]

We should change element::isBlockBoundary method, as it's also used in other places.

[Martin]

Purpose of patch.

[Garry Yao]

Well, it looks unreasonable to determinate display mode from elements structure.

[Tobiasz Cudnik]

The problem with an attribute migration is solved by second patch. Although on Webkit there's created a new, empty LI after each list & unlinst operations. When we remove the image from the TC, this behavior is gone, so it may be related.

[Garry Yao]

Replying to tobiasz.cudnik:

The problem with an attribute migration is solved by second patch. Although on Webkit there's created a new, empty LI after each list & unlinst operations. When we remove the image from the TC, this behavior is gone, so it may be related.

WFM on webkit with the ticket test, what exactly do you see there?

[Tobiasz Cudnik]

1. Use the following content with the selection:

   [<p>
   	paragraph 1 with float left image <img src="http://a.cksource.com/c/1/inc/img/logo-ckfinder.gif" style="float: left;" /></p>
   <p>
   	paragraph 2</p>
   <p>
   	paragraph 3</p>]
   

2. Apply the List command.
3. Apply the List command.

Expected: Same as in a point 1.

Result:

<p>
	paragraph 1 with float left image</p>
<p>
	<img src="http://a.cksource.com/c/1/inc/img/logo-ckfinder.gif" style="float: left;" /></p>
<p>
	paragraph 2</p>
<p>
	paragraph 3</p>

[Tobiasz Cudnik]

Ok, it's fine, i had a wrongly patched version.

[Garry Yao]

Fixed with [6113].