Ticket #7498: 7498.patch

File 7498.patch, 4.4 KB (added by Garry Yao, 10 years ago)
  • _source/plugins/colorbutton/plugin.js

     
    121121                                        editor.fire( 'saveSnapshot' );
    122122
    123123                                        // Clean up any conflicting style within the range.
    124                                         new CKEDITOR.style( config['colorButton_' + type + 'Style'], { color : 'inherit' } ).remove( editor.document );
     124                                        new CKEDITOR.style( CKEDITOR.tools.extend( config['colorButton_' + type + 'Style'], { splitOnRemove : 1 } ), { color : 'inherit' } ).remove( editor.document );
    125125
    126126                                        if ( color )
    127127                                        {
  • _source/plugins/styles/plugin.js

     
    654654                range.enlarge( CKEDITOR.ENLARGE_ELEMENT, 1 );
    655655
    656656                var bookmark = range.createBookmark(),
    657                         startNode = bookmark.startNode;
     657                        startNode = bookmark.startNode,
     658                        endNode = bookmark.endNode,
     659                        me = this;
    658660
     661                /*
     662                 * Find out the style ancestor that needs to be broken down at startNode
     663                 * and endNode.
     664                 */
     665                function breakNodes()
     666                {
     667                        var startPath = new CKEDITOR.dom.elementPath( startNode.getParent() ),
     668                                endPath = endNode && new CKEDITOR.dom.elementPath( endNode.getParent() ),
     669                                breakStart = null,
     670                                breakEnd = null;
     671                        for ( var i = 0 ; i < startPath.elements.length ; i++ )
     672                        {
     673                                var element = startPath.elements[ i ];
     674
     675                                if ( element == startPath.block || element == startPath.blockLimit )
     676                                        break;
     677
     678                                if ( me.checkElementRemovable( element ) )
     679                                        breakStart = element;
     680                        }
     681                        for ( i = 0 ; endPath && i < endPath.elements.length ; i++ )
     682                        {
     683                                element = endPath.elements[ i ];
     684
     685                                if ( element == endPath.block || element == endPath.blockLimit )
     686                                        break;
     687
     688                                if ( me.checkElementRemovable( element ) )
     689                                        breakEnd = element;
     690                        }
     691
     692                        if ( breakEnd )
     693                                endNode.breakParent( breakEnd );
     694                        if ( breakStart )
     695                                startNode.breakParent( breakStart );
     696                }
     697
    659698                if ( range.collapsed )
    660699                {
    661700
     
    688727                                                boundaryElement = element;
    689728                                                boundaryElement.match = isStart ? 'start' : 'end';
    690729                                        }
     730                                        // If presents collapsed range will split the current style element but not remove the whole element.
     731                                        else if ( this._.definition.splitOnRemove )
     732                                                breakNodes();
    691733                                        else
    692734                                        {
    693735                                                /*
     
    734776                         * Now our range isn't collapsed. Lets walk from the start node to the end
    735777                         * node via DFS and remove the styles one-by-one.
    736778                         */
    737                         var endNode = bookmark.endNode,
    738                                 me = this;
    739 
    740                         /*
    741                          * Find out the style ancestor that needs to be broken down at startNode
    742                          * and endNode.
    743                          */
    744                         function breakNodes()
    745                         {
    746                                 var startPath = new CKEDITOR.dom.elementPath( startNode.getParent() ),
    747                                         endPath = new CKEDITOR.dom.elementPath( endNode.getParent() ),
    748                                         breakStart = null,
    749                                         breakEnd = null;
    750                                 for ( var i = 0 ; i < startPath.elements.length ; i++ )
    751                                 {
    752                                         var element = startPath.elements[ i ];
    753 
    754                                         if ( element == startPath.block || element == startPath.blockLimit )
    755                                                 break;
    756 
    757                                         if ( me.checkElementRemovable( element ) )
    758                                                 breakStart = element;
    759                                 }
    760                                 for ( i = 0 ; i < endPath.elements.length ; i++ )
    761                                 {
    762                                         element = endPath.elements[ i ];
    763 
    764                                         if ( element == endPath.block || element == endPath.blockLimit )
    765                                                 break;
    766 
    767                                         if ( me.checkElementRemovable( element ) )
    768                                                 breakEnd = element;
    769                                 }
    770 
    771                                 if ( breakEnd )
    772                                         endNode.breakParent( breakEnd );
    773                                 if ( breakStart )
    774                                         startNode.breakParent( breakStart );
    775                         }
    776779                        breakNodes();
    777780
    778781                        // Now, do the DFS walk.
  • _source/plugins/font/plugin.js

     
    6565                                        editor.focus();
    6666                                        editor.fire( 'saveSnapshot' );
    6767
     68                                        // Clean up any conflicting style within the range.
     69                                        new CKEDITOR.style( CKEDITOR.tools.extend( config[ 'fontSize_style' ], { splitOnRemove : 1 } ) , { 'size' : 'inherit' } ).remove( editor.document );
     70
    6871                                        var style = styles[ value ];
    6972
    7073                                        if ( this.getValue() == value )
© 2003 – 2021 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy