[IE8 bug] getSelectedText returning empty string when selecting text within an ordered list in a table

[ckeditorTest]

I'm trying to put an anchor tag on a list item embedded in a table. As noted in the summary this bug is happening only in IE8 for me. Works in FF & Chrome. I'm using a custom plugin to get the selected text and set the anchor.

HTML:

<table>
	<tbody>
		<tr>
			<td>
			<ul>
				<li>test1 test</li>
				<li>test2 test</li>
			</ul>
			</td>
		</tr>
	</tbody>
</table>

JS:

		var CKEDITOR = parent.window.opener.CKEDITOR;
		var editor1 = CKEDITOR.instances.editor1;
		var selection = editor1.getSelection();
		var selectedText = selection.getSelectedText();
 
 
		if (selection.getType() == CKEDITOR.SELECTION_ELEMENT) {
		  var selectedContent = selection.getSelectedElement().$.outerHTML;
		} else if (selection.getType() == CKEDITOR.SELECTION_TEXT) {
		  if (CKEDITOR.env.ie) {
		    selection.unlock(true);
		    selectedContent = selection.getNative().createRange().htmlText;
		  } else {
		    selectedContent = selection.getNative();
		  }
		}

[ckeditorTest]

replaced attachment so that files are html instead of asp. I'm only able to reproduce this issue if I use the plugin.

[Jakub Ś]

I got same results as in #12476. Please see ​comment:9 for more details.

For me this issue is also invalid.

[ckeditorTest]

I am able to reproduce this bug in IE9 & IE11 on Windows7 so I think it's a separate issue.

[Jakub Ś]

I have tested both issues and got same results. What is more the code for both bugs is the same (this one has extra plugin but it doesn't change much ).

There is one thing I can recommend: focus editor befiore getting the selection. Use editor.focus().

In standard dialogs we:

• declare variable in init
• set it in {{onShow}}}
• do stuff
• on onOk we use selection saved in variable and apply styles to it.

Since you are using custom dialog, please try focusing editor after opening it and then getting the selection.

[ckeditorTest]

Can I reopen this issue? I think the issue is within the plugin code. I do not see the bug if I insert the link from a regular popup window. Also, the same code works if the html is NOT within a table.

I've provide 2 sets of files - one with the plugin and one without. To reproduce the issue:

1. Open the plugin version
2. Select the 'test' text within the table html tags
3. Click the file manager plugin button
4. Click 'Insert Link' button
5. No link created

Repeat steps above, but this time select the 'test' text outside of the table tags. Link should get created. Repeat steps above with the other sample files that don't use the plugin. Link should get created in all cases.

[ckeditorTest]

I get the same results testing IE8 and IE11 on Win7.

[Jakub Ś]

I have looked at your code but haven't checked it because you haven't tried anything I have proposed. You haven't tried to focus editor or using styles to insert links. Instead you stay with the same code and keep writing that there is an error in editor.

[ckeditorTest]

I have tried to set the focus, but this doesn't seem to be the problem. I've also tried to get styles to insert links, but am not able to get an instance. Not sure how this would even work. I'm trying to replace selected text within a <td>. eg. replace "<td>text</td>" with "<td><a href="​http://test.doc">text</a></td>" I keep getting the error "Invalid procedure call or argument" when I attempt to create an instance of the style object. Here's my code:

var CKEDITOR = parent.window.opener.CKEDITOR;
var editor1 = CKEDITOR.instances.editor1;
var selection = editor1.getSelection();
var range = selection.getRanges()[0];
var style = new CKEDITOR.style( { element: 'td', attributes: { 'class': 'foo' } } );

style.type = CKEDITOR.STYLE_INLINE;
style.applyToRange( range, editor1 );
range.select();

I go back to the question as to why this seems to be only an issue within tables for IE?

[Jakub Ś]

I have checked your code and made some modifications but for some weird reason it was throwing JavaScript error in IE8.

One thing is for sure - I remember this problem from IE and CKEditor 3.x and the best way to fix it was focusing editor and using styles.
Taking all the above into account please see attached code with solution and a little bit different dialog.

NOTE: I have checked this with CKEditor 4.4.5