#2820: Forms plugin
-------------------------+--------------------------------------------------
 Reporter:  arczi        |       Owner:  arczi       
     Type:  New Feature  |      Status:  new         
 Priority:  Normal       |   Milestone:  CKEditor 3.0
Component:  General      |     Version:              
 Keywords:               |  
-------------------------+--------------------------------------------------

Comment(by fredck):

 The patch is almost good. It misses the changes to the "select" dialog
 though.

 Also, the reason for this changes is to have the element logic attached to
 the element itself. In this way, the dialog continue working even if we
 remove one of the elements. For that, we need to make the onShow and onOk
 implementations quite generic, independent of specific elements. So, calls
 like getContentElement must be avoided inside those functions.

 I've noted that most of the dialogs are still using getContentElement
 inside onShow, to set the startup element focus. Other than the problem
 I've just described, this is something that must not be handled by the
 dialog definition, but by the API itself, which should automatically set
 the caret position.

 So, a new patch should be provided without those getContentElement calls.

-- 
Ticket URL: <http://dev.fckeditor.net/ticket/2820#comment:1>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
This SF.net email is sponsored by:
SourcForge Community
SourceForge wants to tell your story.
http://p.sf.net/sfu/sf-spreadtheword
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac

Reply via email to