#3728: Automatic color options don't seem to do anything
-----------------------------------+----------------------------------------
 Reporter:  damo                   |       Owner:  martinkou   
     Type:  Bug                    |      Status:  assigned    
 Priority:  Normal                 |   Milestone:  CKEditor 3.0
Component:  General                |     Version:              
 Keywords:  IBM Confirmed Review-  |  
-----------------------------------+----------------------------------------
Changes (by garry.yao):

  * keywords:  IBM Confirmed Review? => IBM Confirmed Review-


Comment:

 I think you fixing is somehow conflicting with the existing logic there:
 {{{

                                         if ( color )
                                                 style.apply(
 editor.document );
                                         else
                                                 style.remove(
 editor.document );

 }}}
 So the correct fix would either remove the current colored style OR apply
 the inherited style.

-- 
Ticket URL: <http://dev.fckeditor.net/ticket/3728#comment:4>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables unlimited
royalty-free distribution of the report engine for externally facing 
server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
FCKeditor-Trac mailing list
FCKeditor-Trac@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac

Reply via email to