Changeset 4838


Ignore:
Timestamp:
12/29/09 15:16:37 (4 years ago)
Author:
garry.yao
Message:

#4892: Apply table align without text surrounding by first apply alignment attribute to the table element and following a div with clear float style.

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r4837 r4838  
    7777                <li><a href="http://dev.fckeditor.net/ticket/4905">#4905</a> : Fixed paste plain text result incorrect when content from dialog.</li> 
    7878                <li><a href="http://dev.fckeditor.net/ticket/4889">#4889</a> : Fixed unable to undo 'New Page' command after typing inside editor.</li> 
     79                <li><a href="http://dev.fckeditor.net/ticket/4892">#4892</a> : Fixed table alignment style is not properly represented by the wrapping div.</li> 
    7980        </ul> 
    8081        <h3> 
  • CKEditor/trunk/_source/plugins/pastefromword/filter/default.js

    r4818 r4838  
    766766                                        }, 
    767767 
     768                                        'div' : function( element ) 
     769                                        { 
     770                                                // Aligned table with no text surrounded is represented by a wrapper div, from which 
     771                                                // table cells inherit as text-align styles, which is wrong. 
     772                                                // Instead we use a clear-float div after the table to properly achieve the same layout. 
     773                                                var singleChild = element.onlyChild(); 
     774                                                if( singleChild && singleChild.name == 'table' ) 
     775                                                { 
     776                                                        var attrs = element.attributes; 
     777                                                        singleChild.attributes = CKEDITOR.tools.extend( singleChild.attributes, attrs ); 
     778                                                        attrs.style && singleChild.addStyle( attrs.style ); 
     779 
     780                                                        var clearFloatDiv = new CKEDITOR.htmlParser.element( 'div' ); 
     781                                                        clearFloatDiv.addStyle( 'clear' ,'both' ); 
     782                                                        element.add( clearFloatDiv ); 
     783                                                        delete element.name; 
     784                                                } 
     785                                        }, 
     786 
    768787                                        'td' : function ( element ) 
    769788                                        { 
     
    945964                                                                } 
    946965                                                        } ], 
     966 
     967                                                // Preserve clear float style. 
     968                                                [ /^clear$/ ], 
    947969 
    948970                                                [ ( /^border.*|margin.*|vertical-align|float$/ ), null, 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy