Ticket #1232 (closed Bug: fixed)

Opened 7 years ago

Last modified 7 years ago

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

comment:1 Changed 7 years ago by fredck

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

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 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy