Changeset 6958 for CKEditor/trunk


Ignore:
Timestamp:
05/24/2011 03:43:30 PM (4 years ago)
Author:
Saare
Message:

#7158: Iframe dialog: width and height fields are now synced with the style field.

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r6957 r6958  
    7474                <li><a href="http://dev.ckeditor.com/ticket/6663">#6663</a> : Table caption that contains rich text is not anymore corrupted after edit with table dialog.</li> 
    7575                <li><a href="http://dev.ckeditor.com/ticket/7192">#7192</a> : [IE] Selecting paragrpah format in an empty editor without focus didn't bring focus into editor body.</li> 
     76                <li><a href="http://dev.ckeditor.com/ticket/7158">#7158</a> : Width and height fields in the iframe dialog are now synchronized with the style field.</li> 
    7677                <li>Updated the following language files:<ul> 
    7778                        <li><a href="http://dev.ckeditor.com/ticket/7834">#7834</a> : Dutch;</li> 
  • CKEditor/trunk/_source/plugins/iframe/dialogs/iframe.js

    r6348 r6958  
    4949                        minWidth : 350, 
    5050                        minHeight : 260, 
     51                        onLoad : function() 
     52                        { 
     53                                var dialog = this, 
     54                                        styles = dialog.getContentElement( 'advanced', 'advStyles' ); 
     55 
     56                                styles && styles.on( 'change', function() 
     57                                        { 
     58                                                // Synchronize width value. 
     59                                                var width = parseInt( this.getStyle( 'width', '' ) || '', 10 ), 
     60                                                        txtWidth = dialog.getContentElement( 'info', 'width' ); 
     61 
     62                                                txtWidth && txtWidth.setValue( width, true ); 
     63 
     64                                                // Synchronize height value. 
     65                                                var height = parseInt( this.getStyle( 'height', '' ) || '', 10 ), 
     66                                                        txtHeight = dialog.getContentElement( 'info', 'height' ); 
     67 
     68                                                txtHeight && txtHeight.setValue( height, true ); 
     69                                        }); 
     70                        }, 
    5171                        onShow : function() 
    5272                        { 
     
    6484                                        this.setupContent( iframeNode, fakeImage ); 
    6585                                } 
     86 
     87                                // Call the onChange method for the widht and height fields so 
     88                                // they get reflected into the Advanced tab. 
     89                                var widthInput = this.getContentElement( 'info', 'width' ), 
     90                                        heightInput = this.getContentElement( 'info', 'height' ); 
     91                                widthInput && widthInput.onChange(); 
     92                                heightInput && heightInput.onChange(); 
    6693                        }, 
    6794                        onOk : function() 
     
    140167                                                                                if ( this.getValue() ) 
    141168                                                                                        extraStyles.width = this.getValue() + 'px'; 
     169                                                                        }, 
     170                                                                        onChange : function() 
     171                                                                        { 
     172                                                                                var styles = this.getDialog().getContentElement( 'advanced', 'advStyles' ), 
     173                                                                                        value = this.getValue(); 
     174                                                                                styles && styles.updateStyle( 'width', value &&  ( value + 'px' ) ); 
    142175                                                                        } 
    143176                                                                }, 
     
    164197                                                                                if ( this.getValue() ) 
    165198                                                                                        extraStyles.height = this.getValue() + 'px'; 
     199                                                                        }, 
     200                                                                        onChange : function() 
     201                                                                        { 
     202                                                                                var styles = this.getDialog().getContentElement( 'advanced', 'advStyles' ), 
     203                                                                                        value = this.getValue(); 
     204                                                                                styles && styles.updateStyle( 'height', value &&  ( value + 'px' ) ); 
    166205                                                                        } 
    167206                                                                }, 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy