Changeset 5882


Ignore:
Timestamp:
09/14/10 19:37:54 (4 years ago)
Author:
fredck
Message:

#6192, #6178, #6179: [WebKit] Insertion of elements in the editor was failing if the editor didn't receive the focus first.

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r5881 r5882  
    8484                <li><a href="http://dev.ckeditor.com/ticket/6075">#6075</a> : [FF] Newly created links didn't fill in information on edit.</li> 
    8585                <li><a href="http://dev.ckeditor.com/ticket/6183">#6183</a> : The toolbar panels options sometimes had the contents' link color.</li> 
     86                <li><a href="http://dev.ckeditor.com/ticket/6192">#6192</a> : [WebKit] Inserting smileys was not working because of editor focus issues.</li> 
     87                <li><a href="http://dev.ckeditor.com/ticket/6178">#6178</a> : [WebKit] Inserting elements by code was failing if the editor didn't receive the focus first.</li> 
     88                <li><a href="http://dev.ckeditor.com/ticket/6179">#6179</a> : [WebKit] The Image dialog was not working if the editor didn't receive the focus first.</li> 
    8689                <li>Updated the following language files:<ul> 
    8790                        <li><a href="http://dev.ckeditor.com/ticket/6246">#6246</a> : Chinese Simplified;</li> 
  • CKEditor/trunk/_source/plugins/selection/plugin.js

    r5877 r5882  
    692692                                        // instances. 
    693693 
    694                                         var ranges = []; 
    695                                         var sel = this.getNative(); 
     694                                        var ranges = [], 
     695                                                range, 
     696                                                doc = this.document, 
     697                                                sel = this.getNative(); 
    696698 
    697699                                        if ( !sel ) 
    698                                                 return []; 
     700                                                return ranges; 
     701 
     702                                        // On WebKit, it may happen that we'll have no selection 
     703                                        // available. We normalize it here by replicating the 
     704                                        // behavior of other browsers. 
     705                                        if ( !sel.rangeCount ) 
     706                                        { 
     707                                                range = new CKEDITOR.dom.range( doc ); 
     708                                                range.moveToElementEditStart( doc.getBody() ); 
     709                                                ranges.push( range ); 
     710                                        } 
    699711 
    700712                                        for ( var i = 0 ; i < sel.rangeCount ; i++ ) 
    701713                                        { 
    702714                                                var nativeRange = sel.getRangeAt( i ); 
    703                                                 var range = new CKEDITOR.dom.range( this.document ); 
     715 
     716                                                range = new CKEDITOR.dom.range( doc ); 
    704717 
    705718                                                range.setStart( new CKEDITOR.dom.node( nativeRange.startContainer ), nativeRange.startOffset ); 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy