Changeset 7004 for CKEditor/trunk


Ignore:
Timestamp:
06/02/2011 05:26:36 PM (3 years ago)
Author:
garry.yao
Message:

#7959: Place cursor in first table cell after table insertion.

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r7003 r7004  
    4747                <li><a href="http://dev.ckeditor.com/ticket/7387">#7387</a> : Allow styleDefinitions that can be applied to a set of elements.</li> 
    4848                <li><a href="http://dev.ckeditor.com/ticket/4345">#4345</a> : Added the new "langLoaded" event to CKEDITOR.editor so "by code" language updates can be performed.</li> 
     49                <li><a href="http://dev.ckeditor.com/ticket/7959">#7959</a> : Table insertion will blink cursor in the first cell.</li> 
    4950        </ul> 
    5051        <p> 
  • CKEditor/trunk/_source/plugins/table/dialogs/table.js

    r6987 r7004  
    107107                        { 
    108108                                var selection = editor.getSelection(), 
    109                                         bms = selection.createBookmarks(); 
     109                                        bms = this._.selectedElement && selection.createBookmarks(); 
    110110 
    111111                                var table = this._.selectedElement || makeElement( 'table' ), 
     
    223223                                // Insert the table element if we're creating one. 
    224224                                if ( !this._.selectedElement ) 
     225                                { 
    225226                                        editor.insertElement( table ); 
    226  
     227                                        // Override the default cursor position after insertElement to place 
     228                                        // cursor inside the first cell (#7959), IE needs a while. 
     229                                        setTimeout( function() 
     230                                                { 
     231                                                        var firstCell = new CKEDITOR.dom.element( table.$.rows[ 0 ].cells[ 0 ] ); 
     232                                                        var range = new CKEDITOR.dom.range( editor.document ); 
     233                                                        range.moveToPosition( firstCell, CKEDITOR.POSITION_AFTER_START ); 
     234                                                        range.select( 1 ); 
     235                                                }, 0 ); 
     236                                } 
    227237                                // Properly restore the selection, (#4822) but don't break 
    228238                                // because of this, e.g. updated table caption. 
    229                                 try { selection.selectBookmarks( bms ); } catch( er ){} 
    230  
    231                                 return true; 
     239                                else 
     240                                        try { selection.selectBookmarks( bms ); } catch( er ){} 
    232241                        }, 
    233242                        contents : [ 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy