#3001: IE : Automatically save the selection when loosing focus
-------------------------------+--------------------------------------------
Reporter: fredck | Owner: fredck
Type: Bug | Status: assigned
Priority: High | Milestone: CKEditor 3.0
Component: General | Version:
Keywords: Confirmed Review+ |
-------------------------------+--------------------------------------------
Changes (by garry.yao):
* keywords: Confirmed Review? => Confirmed Review+
Comment:
It's worthy to takes a long time to validate all those changes introduced
by the new patch, it really eliminate all those previous hacks everywhere,
so R+.[[BR]]
Though, I found the following minor issues:
1. It's not necessary to have L132 of
''_source\plugins\selection\plugin.js'', since a saving will be fired by
'selectchange' event at the moment;
1. Within ''selectElement'' method L766, maybe the cached type should be
'CKEDITOR.SELECTION_ELEMENT' instead.
1. Within ''selectRanges'' method L813, we can't leave 'selectedElement'
to null, otherwise later it will be retrieved from native selection, which
is error prone, also the 'type' and 'startElement' is not accurately
calculated.
--
Ticket URL: <http://dev.fckeditor.net/ticket/3001#comment:7>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac
