#3152: basicstyle override tag not working
-------------------------------+--------------------------------------------
 Reporter:  garry.yao          |       Owner:  garry.yao      
     Type:  Bug                |      Status:  assigned       
 Priority:  Normal             |   Milestone:  CKEditor 3.0   
Component:  General            |     Version:  SVN (FCKeditor)
 Keywords:  Confirmed Review?  |  
-------------------------------+--------------------------------------------
Changes (by garry.yao):

  * keywords:  Confirmed Review- => Confirmed Review?


Comment:

 Replying to [comment:6 fredck]:
 > Still a new issues:
 >
 >  * The cache trick at line 936 has no effect.
 Yes, it's a big mistake.
 >  * If you look at the colobutton plugin, "colorButton_foreStyle" defines
 a style that has an override for the <font> element, containing the
 "color" attribute (null means "any value" there). It means that if you
 have the following:
 >
 > {{{
 > <font color="red">Test</font>
 > }}}
 >
 > Doing a "select all" and applying a new color, the <font> tag should
 disappear. What we have instead is the following:
 >
 > {{{
 > <span style="color: rgb(0, 0, 255);"><font
 color="red">Test</font></span>
 > }}}
 I found the issued was caused by a deeper hiding bug from #3087, which
 should not be fixed at this ticket, verified by the following codes:
 {{{
 CKEDITOR.tools.isArray( new CKEDITOR.style(
 CKEDITOR.instances.editor1.config.colorButton_foreStyle.overrides
 )._.definition.overrides )
 }}}

-- 
Ticket URL: <http://dev.fckeditor.net/ticket/3152#comment:7>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
This SF.net email is sponsored by:
High Quality Requirements in a Collaborative Environment.
Download a free trial of Rational Requirements Composer Now!
http://p.sf.net/sfu/www-ibm-com
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac

Reply via email to