CKE 3.x and 4.x - Undefined table width gets reset to default

[Jakub Ś]

This is the continuation of #9478.

To reproduce:

1. Clear editor contents with CRTL+A Backspace
2. Insert default table
3. Open table properties dialog, clear width value and click OK
4. Open table properties one more time.

Result: Default table width 500px is present in width field and once you click OK table gets default width.

You have to use below code to see the bug

CKEDITOR.on( 'dialogDefinition', function( ev )
            {
                var dialogName = ev.data.name;
                var dialogDefinition = ev.data.definition;

                // Set default table width during creation.
                if(dialogName == 'table')
                {
                    var infoTab = dialogDefinition.getContents( 'info' );
                    txtWidth = infoTab.get( 'txtWidth' );
                    txtWidth['default'] = '100%';
                }

                if ( dialogName == 'tableProperties' )
                {
                    var infoTab = dialogDefinition.getContents( 'info' );

                    var tab = dialogDefinition.getContents( 'advanced' );
                    tab.remove ('advLangDir');
                    tab.remove ('advStyles');
                    tab.remove ('advCSSClasses');
                }
            });

window.x  = CKEDITOR.editor.replace('editor1', {
    removePlugins : 'button,div,filebrowser,flash,format,forms,horizontalrule,indent,justify,liststyle,pagebreak,showborders,stylescombo,templates',
        toolbar :
            [
                ['Source', 'Table']
            ],
});

With this code bug can be reproduced in v4 but also in v3 from CKE 3.6.1 rev. [6979]

[Joel]

I think I managed to narrow down even further. The line that actually causes the bug to appear is tab.remove('advStyles'). I updated a jsfiddle to v4 for this: ​http://jsfiddle.net/gz9b4/4/ - the code for replicating in 3.6.2 at least is as follows

 CKEDITOR.on(
    'dialogDefinition', 
    function( ev )
    {
        var dialogName = ev.data.name;
        var dialogDefinition = ev.data.definition;

        if ( dialogName == 'tableProperties' )
        {
            var tab = dialogDefinition.getContents( 'advanced' );
            tab.remove ('advStyles'); // This causes the bug to appear
            //tab.remove ('advLangDir'); // this does not
            //tab.remove ('advCSSClasses'); // nor this
        }
    }
);
window.x  = CKEDITOR.editor.replace('my-editor');