Re: [FCKeditor-Trac] [FCKeditor Dev] #4048: Context submenu lacks uiColor

[<fck...@li...>]

#4048: Context submenu lacks uiColor
------------------------------------+---------------------------------------
 Reporter:  tobiasz.cudnik          |       Owner:  tobiasz.cudnik
     Type:  Bug                     |      Status:  assigned      
 Priority:  Normal                  |   Milestone:  CKEditor 3.1  
Component:  General                 |     Version:                
 Keywords:  Kama Confirmed Review-  |  
------------------------------------+---------------------------------------
Changes (by garry.yao):

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


Comment:

 The patch works for me in all browsers, while there're some issues noticed
 in codes:
  1. L160 - L170 of skin.js could be merged into 'updateStylesheets'
 function;
  1. It's better to use RegExp for global replacement;
  1. L176 - L192 should be merged within the guard condition at L195.

-- 
Ticket URL: <http://dev.fckeditor.net/ticket/4048#comment:7>
CKEditor <http://ckeditor.com/>
The text editor for Internet