Changeset 5962


Ignore:
Timestamp:
10/20/10 07:17:23 (3 years ago)
Author:
garry.yao
Message:

#6083: Selection restoring logic, might normalize text nodes beneath thus potentially invalidate the selection lock, avoid using them together.

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r5961 r5962  
    7474                <li><a href="http://dev.ckeditor.com/ticket/5766">#5766</a> : Config entry "ignoreEmptyParagraph" should only remove one single empty paragraph in document.</li> 
    7575                <li><a href="http://dev.ckeditor.com/ticket/5931">#5931</a> : Unable to apply inline style because of nested elements with same style name.</li> 
     76                <li><a href="http://dev.ckeditor.com/ticket/6083">#6083</a> : Dialog close sometimes cause collapsed editor selection before the insertion.</li> 
    7677                <li>Updated the following language files:<ul> 
    7778                        <li><a href="http://dev.ckeditor.com/ticket/6427">#6427</a> : Ukrainian;</li> 
  • CKEditor/trunk/_source/plugins/selection/plugin.js

    r5949 r5962  
    134134                                                                if ( savedRange ) 
    135135                                                                { 
    136                                                                         if ( restoreEnabled ) 
     136                                                                        // Range restored here might invalidate the DOM structure thus break up 
     137                                                                        // the locked selection, give it up. (#6083) 
     138                                                                        var lockedSelection = doc.getCustomData( 'cke_locked_selection' ); 
     139                                                                        if ( restoreEnabled && !lockedSelection ) 
    137140                                                                        { 
    138141                                                                                // Well not break because of this. 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy