Changeset 6958 for CKEditor/trunk


Ignore:
Timestamp:
05/24/11 15:43:30 (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 – 2015 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy