insertElement incorrect after deleteContents

[Garry Yao]

If there's a insertion happened after the selection range content is deleted, the inserted element is at the end instead of in the front.
Take the smiley plugin for reproducing:

1. Make the content and selection as below:

   		<p>te^xt</p>
   		<ul>
   			<li>te^xt</li>
   		</ul>
   
   

2. Open the smiley plugin to insert a motion;

• Expected Result:

  <p>te<img alt=":)" title=":)" ...></p><ul><li>xt</li></ul>
  

• Actual Result:

  <p>te</p><img alt=":)" title=":)" ...><ul><li>xt</li></ul>
  

[Frederico Caldeira Knabben]

I think you have inverted the "expected" and "actual" results, right?

[Garry Yao]

Updated the bug symptom with current trunk.

[Frederico Caldeira Knabben]

Please do not change the ticket description in this way. If anything changed, you can add comments for it.

In this specific case for example, your original ticket text was much more useful to understand the bug (the js error is just another issue here).

[Garry Yao]

Replying to fredck:

Please do not change the ticket description in this way. If anything changed, you can add comments for it.

In this specific case for example, your original ticket text was much more useful to understand the bug (the js error is just another issue here).

Revert to the old description.

[Garry Yao]

The proposed path will insert those inline elements at the original editable end after deletion instead of the collapsed position.

[Garry Yao]

The correct carot position should be at the original editable start instead.

[Garry Yao]

The below line might be confusing since it's relay on the buggy 'getBoundaryNodes', which report the result nodes up-side-down on collapsed range(#3292).

[Garry Yao]

Replying to garry.yao:

// Place the editing carot after the editing end of previous node.
range.moveToElementEditEnd( range.getBoundaryNodes().endNode );

[Tobiasz Cudnik]

Please refresh patch to newest trunk, as there are conflicts right now. Also, the patch filename contains wrong ticket number.

[Garry Yao]

Ticket Test added at :
​http://ckeditor.t/tt/3313/1.html.

[Garry Yao]

Ticket Test updated at : ​http://ckeditor.t/tt/3133/1.html[[BR]]

[Frederico Caldeira Knabben]

This patch needs updates again, as it's not anymore "applyable" into trunk.

[Frederico Caldeira Knabben]

Milestone CKEditor 3.x deleted