Changeset 6320


Ignore:
Timestamp:
01/14/2011 05:24:42 PM (4 years ago)
Author:
garry.yao
Message:

#6860: double click behavior correction in FF on read-only elements.

Location:
CKEditor/trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r6319 r6320  
    9292                <li><a href="http://dev.ckeditor.com/ticket/6639">#6639</a> :  Line-breaks inside pasted list item from MS-Word breaks list structure.</li> 
    9393                <li><a href="http://dev.ckeditor.com/ticket/6909">#6909</a> :  [IE] GIF icons of toolbar button from custom plugins are not diplayed in zoom level 100%.</li> 
     94                <li><a href="http://dev.ckeditor.com/ticket/6860">#6860</a> :  [FF] double click on placeholder to open dialog throws JavaScript error.</li> 
    9495        </ul> 
    9596        <h3> 
  • CKEditor/trunk/_source/plugins/placeholder/dialogs/placeholder.js

    r6241 r6320  
    4949                        { 
    5050                                if ( isEdit ) 
    51                                 { 
    52                                         var range = editor.getSelection().getRanges()[0]; 
    53                                         range.shrink( CKEDITOR.SHRINK_TEXT ); 
    54                                         var node = range.startContainer; 
    55                                         while( node && !( node.type == CKEDITOR.NODE_ELEMENT && node.data( 'cke-placeholder' ) ) ) 
    56                                                 node = node.getParent(); 
    57                                         this._element = node; 
    58                                 } 
     51                                        this._element = CKEDITOR.plugins.placeholder.getSelectedPlaceHoder( editor ); 
    5952 
    6053                                this.setupContent( this._element ); 
  • CKEditor/trunk/_source/plugins/placeholder/plugin.js

    r6241 r6320  
    5959                        editor.on( 'doubleclick', function( evt ) 
    6060                                { 
    61                                         var element = evt.data.element; 
    62                                         if ( element.data( 'cke-placeholder' ) ) 
     61                                        if ( CKEDITOR.plugins.placeholder.getSelectedPlaceHoder( editor ) ) 
    6362                                                evt.data.dialog = 'editplaceholder'; 
    6463                                }); 
     
    157156                else 
    158157                        editor.insertElement( element ); 
     158        }, 
     159 
     160        getSelectedPlaceHoder : function( editor ) 
     161        { 
     162                var range = editor.getSelection().getRanges()[ 0 ]; 
     163                range.shrink( CKEDITOR.SHRINK_TEXT ); 
     164                var node = range.startContainer; 
     165                while( node && !( node.type == CKEDITOR.NODE_ELEMENT && node.data( 'cke-placeholder' ) ) ) 
     166                        node = node.getParent(); 
     167                return node; 
    159168        } 
    160169}; 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy