#2785: Move plugins specific configurations to the plugins files
-------------------------------+--------------------------------------------
 Reporter:  fredck             |       Owner:  arczi       
     Type:  Bug                |      Status:  new         
 Priority:  High               |   Milestone:  CKEditor 3.0
Component:  General            |     Version:              
 Keywords:  Confirmed Review-  |  
-------------------------------+--------------------------------------------
Changes (by fredck):

  * keywords:  Confirmed Review? => Confirmed Review-


Comment:

 The patch is almost good. Just two small things:

  * The "magnetDistance", "backgroundCoverOpacity" and
 "backgroundCoverColor" configurations must also be moved. They are
 specific to the "dialog" plugin.

  * Code style: All configuration setting statements are missing the
 semicolon after the literal object definition. (e.g. line 38 in
 plugins/table/plugin.js)

 Another detail... please always create the patch from the root of the
 prototype, not the _source folder.

-- 
Ticket URL: <http://dev.fckeditor.net/ticket/2785#comment:3>
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