Changeset 7520


Ignore:
Timestamp:
06/27/12 18:30:09 (2 years ago)
Author:
garry.yao
Message:

#9057: Fixed decrease indent of list item don't need to establish new block.

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r7519 r7520  
    6464                <li><a href="http://dev.ckeditor.com/ticket/8978">#8978</a> : [IE] Saved editor text selection is lost after close menu.</li> 
    6565                <li><a href="http://dev.ckeditor.com/ticket/9022">#9022</a> : [IE7] Fixed wrong selection introduced in empty editor document when click to the bottom-right of the document.</li> 
     66                <li><a href="http://dev.ckeditor.com/ticket/9057">#9057</a> : Unexpected block established when decreasing indent of a styled list item.</li> 
    6667        </ul> 
    6768        <h3> 
  • CKEditor/trunk/_source/plugins/list/plugin.js

    r7508 r7520  
    109109                        while ( 1 ) 
    110110                        { 
    111                                 var item = listArray[ currentIndex ]; 
     111                                var item = listArray[ currentIndex ], 
     112                                        itemGrandParent = item.grandparent; 
    112113 
    113114                                orgDir = item.element.getDirection( 1 ); 
     
    145146                                        currentIndex = listData.nextIndex; 
    146147                                } 
    147                                 else if ( item.indent == -1 && !baseIndex && item.grandparent ) 
    148                                 { 
    149                                         if ( listNodeNames[ item.grandparent.getName() ] ) 
     148                                else if ( item.indent == -1 && !baseIndex && itemGrandParent ) 
     149                                { 
     150                                        if ( listNodeNames[ itemGrandParent.getName() ] ) 
    150151                                                currentListItem = item.element.clone( false, true ); 
    151152                                        else 
     
    154155                                        // Migrate all children to the new container, 
    155156                                        // apply the proper text direction. 
    156                                         var dirLoose = item.grandparent.getDirection( 1 ) != orgDir, 
    157                                                 needsBlock = currentListItem.type == CKEDITOR.NODE_DOCUMENT_FRAGMENT && 
    158                                                                          paragraphMode != CKEDITOR.ENTER_BR, 
     157                                        var dirLoose = itemGrandParent.getDirection( 1 ) != orgDir, 
    159158                                                li = item.element, 
    160159                                                className = li.getAttribute( 'class' ), 
    161160                                                style = li.getAttribute( 'style' ); 
     161 
     162                                        var needsBlock = currentListItem.type == 
     163                                                         CKEDITOR.NODE_DOCUMENT_FRAGMENT && 
     164                                                         ( paragraphMode != CKEDITOR.ENTER_BR || dirLoose || 
     165                                                           style || className ); 
    162166 
    163167                                        var child, count = item.contents.length; 
     
    179183                                                        className && child.addClass( className ); 
    180184                                                } 
    181                                                 else if ( dirLoose || needsBlock || style || className ) 
     185                                                else if ( needsBlock ) 
    182186                                                { 
    183187                                                        // Establish new block to hold text direction and styles. 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy