#2885: Implement the div container command.
-------------------------------+--------------------------------------------
 Reporter:  martinkou          |       Owner:  martinkou      
     Type:  Task               |      Status:  new            
 Priority:  Normal             |   Milestone:  CKEditor 3.0   
Component:  General            |     Version:  SVN (FCKeditor)
 Keywords:  Confirmed Review-  |  
-------------------------------+--------------------------------------------
Changes (by garry.yao):

  * keywords:  Confirmed Review? => Confirmed Review-


Comment:

 1. Debug statement need to be removed on L260 of
 ''_source\plugins\div\dialogs\div.js''
  1. 'compareNodes' function on L272 is idle;
  1. Dialog 'minHeight' config could be reduced, notice that when switch to
 'Advanced' tab, it's able to see unnecessary space after the last field.
  1. Div detection logic on L100 of ''_source\plugins\div\plugin.js'' is
 not right, reproduce with this procedure:[[BR]]
         === Reproducing Procedures ===
         1. Open  the ''replace by class'' example page;
         1. Make the following content with selection:
         {{{
                 <table>
                         <tr><td><div>te^xt</div></td></tr>
                 </table>
         }}}
         1. Right click on the selection position to open context menu;
            * Expected Result: There's div editing items on menu;
            * Actual Result: The div editing items were not on menu;

-- 
Ticket URL: <http://dev.fckeditor.net/ticket/2885#comment:14>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
Stay on top of everything new and different, both inside and 
around Java (TM) technology - register by April 22, and save
$200 on the JavaOne (SM) conference, June 2-5, 2009, San Francisco.
300 plus technical and hands-on sessions. Register today. 
Use priority code J9JMT32. http://p.sf.net/sfu/p
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac

Reply via email to