#3839: SCAYT plug-in update
--------------------------------+-------------------------------------------
 Reporter:  SpellChecker.net    |       Owner:  garry.yao      
     Type:  Bug                 |      Status:  assigned       
 Priority:  Normal              |   Milestone:  CKEditor 3.0   
Component:  UI : Spell Checker  |     Version:  CKEditor 3.0 RC
 Keywords:  Review?             |  
--------------------------------+-------------------------------------------

Comment(by SpellChecker.net):

 patch looks ok.

 one note on the code which set default parameters values:

 CKEDITOR.config.scayt_maxSuggestions =
 CKEDITOR.config.scayt_maxSuggestions || 5;
 CKEDITOR.config.scayt_autoStartup = CKEDITOR.config.scayt_autoStartup ||
 false;

 In your code parameters are hard-coded and not checking whether they are
 already set. Does the config.js processed after plug-ins are loaded?

-- 
Ticket URL: <http://dev.fckeditor.net/ticket/3839#comment:5>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
Enter the BlackBerry Developer Challenge  
This is your chance to win up to $100,000 in prizes! For a limited time, 
vendors submitting new applications to BlackBerry App World(TM) will have
the opportunity to enter the BlackBerry Developer Challenge. See full prize  
details at: http://p.sf.net/sfu/Challenge
_______________________________________________
FCKeditor-Trac mailing list
FCKeditor-Trac@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac

Reply via email to