[IE] Insert select control is causing error

[Garry Yao]

Reproducing Procedures

1. Open the 'replace by class' sample page.
2. Put the cursor among text;
3. Insert a select control;
   • Actual Result: Javascript error.

[Garry Yao]

Fix the incomplete unlocking of IE selection.

[Frederico Caldeira Knabben]

The change looks wrong, as there is no tneed to restore the selection, because there where no changes at all while the dialog has been opened.

Other than that, these changes broke the insertion. For example, the smilieys dialog is not anymore inserting the images in the proper place. You can try it by placing the cursor inside the "using" word and then inserting a smiley.

As a last note... I'm not able to reproduce this issue at all, at least with IE7.

[Garry Yao]

Attracting others to reproduce.

[Tobiasz Cudnik]

I can confirm JS error using IE7 on selection/plugin.js L500, related to nodeValue. Select control have to be inserted between text (there's no error at end of line).

[Artur Formella]

Confirmed. IE 6.0.280

JS error:

nodeValue.length is empty

[Garry Yao]

This's actually a IE7 selection system bug, if you're asking IE to select the Select element of the below content:

select<select></select>element<a href="#">bug</a>

with the following way:

ieNativeRange.moveToElementText( selectElement );

will result a range actually on the <a> element, proved by checking the native range content:

ieNativeRange.textHtml //<a href="#"></a>

Ridiculous indeed, the range text will become the following element's content.
The bug is not happened to IE8, and doesn't happen to other element as I can tell, so I'm proposing not providing fix at all.