Opened 17 years ago

Closed 17 years ago

#1232 closed Bug (fixed)

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

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

Description

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.

Thanks.

Scott

Change History (1)

comment:1 Changed 17 years ago by Frederico Caldeira Knabben

Keywords: delete FCKDomTools GetNextSourceElement removed
Milestone: FCKeditor 2.5
Resolution: fixed
Status: newclosed

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 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy