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