Changeset 5694


Ignore:
Timestamp:
07/12/10 11:48:21 (4 years ago)
Author:
tobiasz.cudnik
Message:

Fixed #5435 IE we can't start Numbered/Bulleted list in Tables by Clicking on Insert/Remove Numbers/Bullets Icon

Location:
CKEditor/trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r5693 r5694  
    4646        <ul> 
    4747                <li><a href="http://dev.fckeditor.net/ticket/5415">#5415</a> : Undo not working when we change the Table Properties for the table on a saved page.</li> 
     48                <li><a href="http://dev.fckeditor.net/ticket/5435">#5435</a> : IE: we can't start Numbered/Bulleted list in Tables by Clicking on Insert/Remove Numbers/Bullets Icon.</li>  
    4849        </ul> 
    4950        <h3> 
  • CKEditor/trunk/_source/plugins/domiterator/plugin.js

    r5206 r5694  
    5252                        { 
    5353                                range = this.range.clone(); 
     54 
     55                                // Shrink the range to exclude harmful "noises" (#4087, #4450, #5435). 
     56                                range.shrink( CKEDITOR.NODE_ELEMENT, true ); 
     57 
    5458                                range.enlarge( this.forceBrBreak || !this.enlargeBr ? 
    5559                                                           CKEDITOR.ENLARGE_LIST_ITEM_CONTENTS : CKEDITOR.ENLARGE_BLOCK_CONTENTS ); 
     
    213217                                // We have found a block boundary. Let's close the range and move out of the 
    214218                                // loop. 
    215                                 if ( ( closeRange || isLast ) && range ) 
    216                                 { 
    217                                         var boundaryNodes = range.getBoundaryNodes(), 
    218                                                 startPath = new CKEDITOR.dom.elementPath( range.startContainer ); 
    219  
    220                                         // Drop the range if it only contains bookmark nodes, and is 
    221                                         // not because of the original collapsed range. (#4087,#4450) 
    222                                         if ( boundaryNodes.startNode.getParent().equals( startPath.blockLimit ) 
    223                                                  && isBookmark( boundaryNodes.startNode ) && isBookmark( boundaryNodes.endNode ) ) 
    224                                         { 
    225                                                 range = null; 
    226                                                 this._.nextNode = null; 
    227                                         } 
    228                                         else 
     219                                if ( isLast || ( closeRange && range ) ) 
    229220                                                break; 
    230                                 } 
    231  
    232                                 if ( isLast ) 
    233                                         break; 
    234  
    235221                        } 
    236222 
  • CKEditor/trunk/_source/plugins/list/plugin.js

    r5456 r5694  
    439439                        { 
    440440                                range = ranges.shift(); 
    441  
    442                                 var boundaryNodes = range.getBoundaryNodes(), 
    443                                         startNode = boundaryNodes.startNode, 
    444                                         endNode = boundaryNodes.endNode; 
    445  
    446                                 if ( startNode.type == CKEDITOR.NODE_ELEMENT && startNode.getName() == 'td' ) 
    447                                         range.setStartAt( boundaryNodes.startNode, CKEDITOR.POSITION_AFTER_START ); 
    448  
    449                                 if ( endNode.type == CKEDITOR.NODE_ELEMENT && endNode.getName() == 'td' ) 
    450                                         range.setEndAt( boundaryNodes.endNode, CKEDITOR.POSITION_BEFORE_END ); 
    451  
    452441                                var iterator = range.createIterator(), 
    453442                                        block; 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy