Changeset 5822


Ignore:
Timestamp:
08/13/10 19:41:53 (3 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 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy