Ticket #4117: 4117_5.patch

File 4117_5.patch, 3.7 KB (added by Garry Yao, 10 years ago)
  • _source/plugins/selection/plugin.js

     
    783783                {
    784784                        if ( this.isLocked )
    785785                        {
    786                                 var range = new CKEDITOR.dom.range();
     786                                var range = new CKEDITOR.dom.range( this.document );
    787787                                range.setStartBefore( element );
    788788                                range.setEndAfter( element );
    789789
  • CHANGES.html

     
    198198                <li><a href="http://dev.fckeditor.net/ticket/3622">#3622</a> : The toolbar in the v2
    199199                        skin was wrapping improperly in some languages.</li>
    200200                <li><a href="http://dev.fckeditor.net/ticket/4119">#4119</a> : Unable to edit image link with image dialog.</li>
     201                <li><a href="http://dev.fckeditor.net/ticket/4117">#4117</a> : Fixed dialog error when transforming image into button.</li>
    201202                <li><a href="http://dev.fckeditor.net/ticket/4058">#4058</a> : [FF] wysiwyg mode is sometimes not been activated.</li>
    202203        </ul>
    203204        <h3>
  • _source/plugins/image/dialogs/image.js

     
    264264                                // Edit existing Image.
    265265                                if ( this.imageEditMode )
    266266                                {
    267                                         var imgTagName = this.imageEditMode,
    268                                                 removeObj = this.imageElement;
     267                                        var imgTagName = this.imageEditMode;
    269268
    270269                                        // Image dialog and Input element.
    271270                                        if ( dialogType == 'image' && imgTagName == 'input' && confirm( editor.lang.image.button2Img ) )
     
    274273                                                imgTagName = 'img';
    275274                                                this.imageElement = editor.document.createElement( 'img' );
    276275                                                this.imageElement.setAttribute( 'alt', '' );
    277                                                 removeObj.insertBeforeMe( this.imageElement );
    278                                                 removeObj.remove( false );
    279 
     276                                                editor.insertElement( this.imageElement );
    280277                                        }
    281278                                        // ImageButton dialog and Image element.
    282279                                        else if ( dialogType != 'image' && imgTagName == 'img' && confirm( editor.lang.image.img2Button ))
     
    290287                                                                alt : ''
    291288                                                        }
    292289                                                );
    293                                                 removeObj.insertBeforeMe( this.imageElement );
    294                                                 removeObj.remove( false );
     290                                                editor.insertElement( this.imageElement );
    295291                                        }
    296292                                }
    297293                                else    // Create a new image.
     
    323319                                                //Insert a new Link.
    324320                                                if ( !this.linkEditMode )
    325321                                                {
    326                                                         this.linkElement.append( this.imageElement, false );
    327                                                         editor.insertElement( this.linkElement );
     322                                                        editor.insertElement(this.linkElement);
     323                                                        this.linkElement.append(this.imageElement, false);
    328324                                                }
    329                                                 else    //Link already exists, image not.
    330                                                         this.linkElement.append( this.imageElement, false );
     325                                                else     //Link already exists, image not.
     326                                                        editor.insertElement(this.imageElement );
    331327                                        }
    332328                                        else
    333329                                                editor.insertElement( this.imageElement );
     
    337333                                        //Add a new link element.
    338334                                        if ( !this.linkEditMode && this.addLink )
    339335                                        {
    340                                                 this.imageElement.insertBeforeMe( this.linkElement );
     336                                                editor.insertElement( this.linkElement );
    341337                                                this.imageElement.appendTo( this.linkElement );
    342338                                        }
    343339                                        //Remove Link, Image exists.
    344340                                        else if ( this.linkEditMode && !this.addLink )
    345                                                 this.linkElement.remove( true );
    346                                 }
     341                                        {
     342                                                editor.getSelection().selectElement( this.linkElement );
     343                                                editor.insertElement( this.imageElement );
     344                                        }
     345                                }
    347346                        },
    348347                        onLoad : function()
    349348                        {
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy