#3408: event handlers are executed in IE and Safari
-------------------------------+--------------------------------------------
Reporter: alfonsoml | Owner: arczi
Type: Bug | Status: assigned
Priority: Normal | Milestone: CKEditor 3.0
Component: General | Version:
Keywords: Confirmed Review- |
-------------------------------+--------------------------------------------
Changes (by fredck):
* keywords: Confirmed Review? => Confirmed Review-
Comment:
The fix is correct, but there is a performance impact on it. We're forcing
it making two loops through the attributes, even if we don't want to sort
them.
This part of the code must be blazing fast. We actually don't use
attribute sorting when loading data into the editor. Even if we'll have a
bit more core because of it, the performance benefits must be considered.
So, instead of changing that "if" condition, we should instead have the
"if ( filter )" case in the else block, inside the "for" loop at line 167.
--
Ticket URL: <http://dev.fckeditor.net/ticket/3408#comment:4>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
Register Now & Save for Velocity, the Web Performance & Operations
Conference from O'Reilly Media. Velocity features a full day of
expert-led, hands-on workshops and two days of sessions from industry
leaders in dedicated Performance & Operations tracks. Use code vel09scf
and Save an extra 15% before 5/3. http://p.sf.net/sfu/velocityconf
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac
