Terminator backspace

[Guillaume Outters]

Summary: Backspace conglomerates blocks, without being smart.

Example: ​http://www.fckeditor.net/demo Source <div><br/><h1>Machin</h1></div> Source click before the « M » backspace

Effect: « Machin » disappears

Cause: fckenterkey.js, line 202 (revision 203): when hitting backspace, except in some rare cases, the current element is merged within the previous one. In the example "crasher", <div><br/><h1>Machin</h1></div> became <div><br>Machin</br></div> oops!

Same with an image, and so on.

Priority: if not for the line asking not to play on Priority on bug reports, I would have set this to High.

[Guillaume Outters]

Quick patch:

 || !previous.innerHTML || !previous.innerHTML.Trim().length

(as in:

if ( previous.nodeName.IEquals( 'TABLE', 'HR' ) || !previous.innerHTML || !previous.innerHTML.Trim().length )

)

The Trim() would even better be replaced by a TotalTrim which would add &nbsp; to the list of blank characters, because user will be puzzled that his second paragraph gets the style of a preceding invisible char (yes, I have some <div>&nbsp;<br/><p>Machin</p></div>…).

[Frederico Caldeira Knabben]

Confirmed with FF2 and IE6, which also throws an error on that case.

[Frederico Caldeira Knabben]

Fixed with [218].

Click here for more info about our SVN system.