Changing paragraph format doesn't work in some situations

[Wiktor Walc]

This is a regression bug, works in CKEditor 3.0.1.

Steps to reproduce:

• open replacebyclass sample
• select the whole first line
• change format to "Formatted"
• expected result: text is surrounded with <pre> tag
• actual result: no changes (<p> tag is still there)

[Frederico Caldeira Knabben]

WFM with both FF3.5 and IE8. Is there any further details on this?

[Wiktor Walc]

I am selecting the text (the whole line) with a mouse in FF 3.5.5, if I select the p tag at the bottom of the editor it works fine. It does not work also if you place the cursor at the beginning and press Shift + End to select the paragraph and then change the format.

[Wiktor Walc]

Hmm actually it has nothing to do with the <pre> tag... the format is simply not changing and it doesn't matter whether it is a heading or something else (I have modified the ticket).

[Frederico Caldeira Knabben]

Ok... true... confirmed with IE8 when using CTRL+A to select all.

[Wiktor Walc]

Apart form problems with changing format when (for example) Ctrl+A is used to select everything, there is also a problem with the blockquote button:

• open IE8
• having the following text:

  <p>
  	This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor</a>.</p>
  

  press Ctrl + A to select everything
• click the blockquote button
• result (in IE8):

  <p>
  	&nbsp;</p>
  <blockquote>
  	<p>
  		</p>
  </blockquote>
  <p>
  	This is some <strong>sample text</strong>. You are using <a href="http://ckeditor.com/">CKEditor</a>.</p>
  

this bug can be also reproduced in FF 3.5.5 when moving cursor at the beginning of line and using Shift+End to select the text instead of pressing Ctrl + A.

I think this is the same bug, just wanted to add another example of reproducing it.

[Frederico Caldeira Knabben]

Regression introduced with [4691].

[Frederico Caldeira Knabben]

The proposed patch reverts some of the changes introduced with [4691] without impacting on #4450.

[Garry Yao]

Design TC updated at : ​http://ckeditor.t/dt/plugins/domiterator/domiterator.html

[Frederico Caldeira Knabben]

Fixed with [4720].