shift+enter may throw error in Editor

[liucougar]

when the selection starts at the beginning of a block element

[liucougar]

In [26217]:

fixes #13697: take into account that a block element may be empty when inserting shift+enter <br>

also fixed a range.deleteContents() issue in IE: it would leave the range invalid in some cases

!strict

[bill]

Thanks. Can you add a test case for this to the EnterKeyHandling.html robot test?

[liucougar]

In [26244]:

refs #13697: added a robot test for this

[bill]

That robot test is failing intermittently for me (tested on IE8).

[liucougar]

In [26663]:

refs #13697: give it more time after keypress before asking for the editor content in EnterKeyHandling test

[Jared Jurkiewicz]

Assuming Liu's change fixed issue, so marking fixed.

[DJ Mountney]

This is still an issue on IE8, and IE7 when blockNodeforEnter='BR'.

<p>ab</p>

If you put the cursor after b, hit ctrl-enter, then backspace, then ctrl-enter again, the browser crashes.

Or if you just enter some text, then use fontchoice to change it's format, then hit ctrl-enter twice at the end of it.

[bill]

I can reproduce a browser crash on IE8 in dijit/tests/editor/EnterKeyHandling.html, by setting the first editor to <p>ab</p>, putting the caret after "b", and then typing ctrl-enter, backspace, ctrl-enter, ctrl-enter. Likewise with shift-enter.

Seems like a different bug than this one though since this is about being at the beginning of a block element. I filed #16832 to track it.