Changeset 5694


Ignore:
Timestamp:
07/12/2010 11:48:21 AM (5 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