Changeset 5822


Ignore:
Timestamp:
08/13/10 19:41:53 (5 years ago)
Author:
fredck
Message:

#6137 : The table dialog was missing the default width value on second opening.

Location:
CKEditor/branches/versions/3.4.x
Files:
5 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/branches/versions/3.4.x/CHANGES.html

    r5821 r5822  
    1 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
     1<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
    22<!--
    33Copyright (c) 2003-2010, CKSource - Frederico Knabben. All rights reserved.
     
    4545                <li><a href="http://dev.fckeditor.net/ticket/6135">#6135</a> : The dialogadvtab plugin now uses the correct label.</li>
    4646                <li><a href="http://dev.fckeditor.net/ticket/6125">#6125</a> : Focus was lost after applying commands in Opera.</li>
     47                <li><a href="http://dev.fckeditor.net/ticket/6137">#6137</a> : The table dialog was missing the default width value on second opening.</li>
    4748        </ul>
    4849        <h3>
  • CKEditor/branches/versions/3.4.x/_source/plugins/dialog/plugin.js

    r5758 r5822  
    763763                        {
    764764                                for ( var j in this._.contents[i] )
    765                                         fn( this._.contents[i][j]);
     765                                        fn( this._.contents[i][j] );
    766766                        }
    767767                        return this;
     
    776776                reset : (function()
    777777                {
    778                         var fn = function( widget ){ if ( widget.reset ) widget.reset(); };
     778                        var fn = function( widget ){ if ( widget.reset ) widget.reset( 1 ); };
    779779                        return function(){ this.foreach( fn ); return this; };
    780780                })(),
     
    23552355                 * Sets the value of this dialog UI object.
    23562356                 * @param {Object} value The new value.
     2357                 * @param {Boolean} noChangeEvent Internal commit, to supress 'change' event on this element.
    23572358                 * @returns {CKEDITOR.dialog.uiElement} The current UI element.
    23582359                 * @example
    23592360                 * uiElement.setValue( 'Dingo' );
    23602361                 */
    2361                 setValue : function( value )
     2362                setValue : function( value, noChangeEvent )
    23622363                {
    23632364                        this.getInputElement().setValue( value );
    2364                         this.fire( 'change', { value : value } );
     2365                        !noChangeEvent && this.fire( 'change', { value : value } );
    23652366                        return this;
    23662367                },
  • CKEditor/branches/versions/3.4.x/_source/plugins/dialogadvtab/plugin.js

    r5819 r5822  
    4343        }
    4444}
    45 
    46 var isUpdating;
    4745
    4846CKEDITOR.plugins.add( 'dialogadvtab',
     
    145143                                                updateStyle : function( name, value )
    146144                                                {
    147                                                         if ( isUpdating )
    148                                                                 return;
    149 
    150                                                         // Flag to avoid recursion.
    151                                                         isUpdating = 1;
    152 
    153145                                                        var styles = this.getValue();
    154146
     
    168160                                                        }
    169161
    170                                                         this.setValue( styles );
    171 
    172                                                         isUpdating = 0;
     162                                                        this.setValue( styles, true );
     163
    173164                                                },
    174165
  • CKEditor/branches/versions/3.4.x/_source/plugins/dialogui/plugin.js

    r5758 r5822  
    6060                },
    6161
    62                 reset : function()
     62                reset : function( noChangeEvent )
    6363                {
    64                         this.setValue( this.getInitValue() );
     64                        this.setValue( this.getInitValue(), noChangeEvent );
    6565                },
    6666
     
    10621062                                {
    10631063                                        !value && ( value = '' );
    1064                                         return CKEDITOR.ui.dialog.uiElement.prototype.setValue.call( this, value );
     1064                                        return CKEDITOR.ui.dialog.uiElement.prototype.setValue.apply( this, arguments );
    10651065                                },
    10661066
     
    11571157                                 * @example
    11581158                                 * @param {Boolean} true to tick the checkbox, false to untick it.
    1159                                  */
    1160                                 setValue : function( checked )
     1159                                 * @param {Boolean} noChangeEvent Internal commit, to supress 'change' event on this element.
     1160                                 */
     1161                                setValue : function( checked, noChangeEvent )
    11611162                                {
    11621163                                        this.getInputElement().$.checked = checked;
    1163                                         this.fire( 'change', { value : checked } );
     1164                                        !noChangeEvent && this.fire( 'change', { value : checked } );
    11641165                                },
    11651166
     
    12231224                                 * @example
    12241225                                 * @param {String} value The value of the button to be chcked.
    1225                                  */
    1226                                 setValue : function( value )
     1226                                 * @param {Boolean} noChangeEvent Internal commit, to supress 'change' event on this element.
     1227                                 */
     1228                                setValue : function( value, noChangeEvent )
    12271229                                {
    12281230                                        var children = this._.children,
     
    12301232                                        for ( var i = 0 ; ( i < children.length ) && ( item = children[i] ) ; i++ )
    12311233                                                item.getElement().$.checked = ( item.getValue() == value );
    1232                                         this.fire( 'change', { value : value } );
     1234                                        !noChangeEvent && this.fire( 'change', { value : value } );
    12331235                                },
    12341236
  • CKEditor/branches/versions/3.4.x/_source/plugins/table/dialogs/table.js

    r5818 r5822  
    3333                        onLoad : function()
    3434                        {
    35                                 var dialog = this,
    36                                         isUpdating;
     35                                var dialog = this;
    3736
    3837                                var styles = dialog.getContentElement( 'advanced', 'advStyles' );
     
    4241                                        styles.on( 'change', function( evt )
    4342                                                {
    44                                                         if ( isUpdating )
    45                                                                 return;
    46 
    47                                                         // Flag to avoid recursion.
    48                                                         isUpdating = 1;
    49 
    5043                                                        // Synchronize width value.
    5144                                                        var width = this.getStyle( 'width', '' ),
     
    6053                                                        }
    6154
    62                                                         txtWidth && txtWidth.setValue( width );
    63                                                         cmbWidthType && cmbWidthType.setValue( isPx ? 'pixels' : 'percents' );
     55                                                        txtWidth && txtWidth.setValue( width, true );
     56                                                        cmbWidthType && cmbWidthType.setValue( isPx ? 'pixels' : 'percents', true );
    6457
    6558                                                        // Synchronize height value.
     
    6861
    6962                                                        height && ( height = parseInt( height, 10 ) );
    70                                                         txtHeight && txtHeight.setValue( height );
    71 
    72                                                         isUpdating = 0;
     63                                                        txtHeight && txtHeight.setValue( height, true );
    7364                                                });
    7465                                }
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2015 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy