The Text and Table template will not copy to the editor text area

[damo]

The Text and Table template will not be copied to the CKEditor text area. This is a problem for IE, only.

Steps to reproduce:

1. IE6, IE7, or IE8.
2. Go to CK Editor nightly build => check the demos => Create and destroy editor instances for Ajax applications.
3. Place the cursor in the CK Editor text area.
4. Click the Template toolbar icon and select Text and Table.

Expected result:
The Text and Table template is copied to the text area, ready for editing.

Actual result:
The Text and Table template appears to be copied to the text area and is then immediately deleted.

Note 1: This scenario works if the browser is FF3 and Safari.

Note 2: If Replace actual contents option is unchecked, this problem does not occur.

[fredck]

Confirmed. It happens with that template only, but with the "Strange Template" the caret is positioned in the wrong place (inside a paragraph added before it automagically).

[tobiasz.cudnik]

Seem to be fixed with [4030]. I would like to ask for one more confirmation.

[garry.yao]

It still happened on IE6 for me, I'm taking over this ticket.

[garry.yao]

E.g. After insert 'Strange Template', document result in:

^<table>....</table> //For IE
<table><tr><td><h3>^....</table> //For Non-IE

After auto fixing for body logics, it becomes the following, which is wicked:

<p><table>....</table>^</p>

The bug is caused by a bug of range::enlarge which was called inside 'onSelectionChangeFixBody' function.

[tobiasz.cudnik]

I'm still getting this error in IE6 strict after applying the patch.

After first template insertion, it's just truncated. On second try - there is a JS error, which screenshot i'm attaching.

[garry.yao]

The second patch is trying to avoid the problem at the first place by manually placing the cursor after each template is applied with insert mode, so the 'onSelectionChangeFixBody' would never bother it.
While the changes from the first patch is still a problem, even a serious one, so it could be also reviewed with this ticket.

[fredck]

There are some conflicts with this patch now. It should be reviewed.

[garry.yao]

Update the patch to trunk.
Note that the range issue would better to be handled by another ticket.

[garry.yao]

Fixed with #4539.