Crash in IE8 when adding an image if there is no selection

[guest]

To reproduce the bug, just try inserting an image when there is no current text selection.

Another clue to the problem can be seen when running demo of the latest build on IE8: http://www.xinha.org/xinha-nightly/examples/ExtendedDemo.html

Simply clicking in the Xinha editor causes an "unspecified exception" at line 752 of InternetExplorer?.js where the call to sel.createRange() fails because sel.type == "None".

-- fas

F. Andy Seidl MyST Technology Partners, Inc.

[guest]

I get this error whenever inserting an image in IE8 when there's no text selected.

I've tried 0.96beta, 0.96beta2, and the most recent nightly.

I've deleted my cache several times, but I seem to continue receiving errors for code that (as far as I can tell) doesn't appear to still be around. I'm a bit perplexed, but perhaps it's just my unfamiliarity with xinha and IE8.

[guest]

Replying to guest:

I get this error whenever inserting an image in IE8 when there's no text selected. I've tried 0.96beta, 0.96beta2, and the most recent nightly. I've deleted my cache several times, but I seem to continue receiving errors for code that (as far as I can tell) doesn't appear to still be around. I'm a bit perplexed, but perhaps it's just my unfamiliarity with xinha and IE8.

I've investigated this further today and it does seem to be a caching issue on my behalf.

It's very strange. IE tells me it clears the cache and has deleted the temp files ,but then insists on loading an old vertsion. I tested this by sticking an alert in there. It loaded the first time today and displayed the alert. I commented it out and saved and deleted my cache. I'm still getting the alert.

Why is IE such a pain?

[guest]

The same error occures when editing an image in IE8. The problem lies in the fact that IE knows two kinds of ranges. TextRanges? and ControlRanges?. When selecting an image and editing it, the code tries to check if the range has been moved by checking if the parentElement has changed. But ControlRanges? don't have a parentElement. I changed the code at rows 473 and below in InternetExplorer?.js ( modules/internetexplorer/ ) from :

if ( findDoc(savedSelection.parentElement()) == findDoc(range.parentElement()))

{

if (range.isEqual(savedSelection)) {

// The selection hasn't moved, no need to restore. return;

}

}

try { savedSelection.select() } catch (e) {}; range = this.createRange(this.getSelection()); if ( range.parentElement() != savedSelection.parentElement())

to

if ( ( savedSelection.parentElement != undefined ) && findDoc(savedSelection.parentElement()) == findDoc(range.parentElement()))

{

if (range.isEqual(savedSelection)) {

// The selection hasn't moved, no need to restore. return;

}

}

try { savedSelection.select() } catch (e) {}; range = this.createRange(this.getSelection()); if (( savedSelection.parentElement != undefined ) && range.parentElement() != savedSelection.parentElement())

which solves the problem.

With regards.

[gogo]

This was fixed in a previous changeset.