#3790: Find & Replace causes "too much recursion"
-----------------------------------+----------------------------------------
Reporter: damo | Owner: garry.yao
Type: Bug | Status: assigned
Priority: Normal | Milestone: CKEditor 3.0
Component: General | Version:
Keywords: IBM Confirmed Review- |
-----------------------------------+----------------------------------------
Changes (by fredck):
* keywords: IBM Confirmed Review? => IBM Confirmed Review-
Comment:
There is definitely a huge performance enhancement with the latest patch.
A more complete test is needed on it, but it works pretty well for me.
There are just small notes regarding the changes on range.js:
* At line 735, we should have the following at this point:
{{{
if ( !collapsed && this.startContainer.equals( this.endContainer ) )
}}}
* The additions at line 775-776 seem unneeded. The range will never be
collapsed at that point, because of the condition at line 748.
* The line space addition at line 724 seems unneeded.
--
Ticket URL: <http://dev.fckeditor.net/ticket/3790#comment:7>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
Are you an open source citizen? Join us for the Open Source Bridge conference!
Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250.
Need another reason to go? 24-hour hacker lounge. Register today!
http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org
_______________________________________________
FCKeditor-Trac mailing list
FCKeditor-Trac@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac
