Opened 8 years ago

Closed 8 years ago

#1232 closed Bug (fixed)

Delete Bug in SVN (likely caused by FCKDomTools.GetNextSourceElement)

Reported by: Scott Owned by:
Priority: Normal Milestone: FCKeditor 2.5 Beta
Component: UI : Enter Key Version: SVN (FCKeditor) - Retired
Keywords: Cc:


I tried testing the SVN version, and found this bug. It occurs the delete key is pressed.

To replicate, set the source to:

<p><strong>strong tag</strong> PRESS DELETE AT THE END OF THIS LINE</p> <p>Another paragraph</p>

Press delete at the end of the first line.

The result is:

<p>PRESS DELETE AT THE END OF THIS LINEstrong tag</p> <p>Another paragraph</p>

I beleive that the first line of problematic code is in fckenterkey.js, FCKEnterKey.prototype.DoDelete at the line:

var eNext = FCKDomTools.GetNextSourceElement( oCurrentBlock, true, [ oRange.StartBlockLimit.nodeName ], ['UL','OL'] ) ;

eNext is the <strong> element in this test. So it makes me think that FCKDomTools.GetNextSourceElement is producing the wrong result.



Change History (1)

comment:1 Changed 8 years ago by fredck

  • Keywords delete FCKDomTools GetNextSourceElement removed
  • Milestone set to FCKeditor 2.5
  • Resolution set to fixed
  • Status changed from new to closed

Scott, you have precisely identified the culprit here... this bug has been introduced with [774].

A silly change with [808] fixed it.

Thanks for the report and for the analysis.

Note: See TracTickets for help on using tickets.
© 2003 – 2015 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy