vanilla ckeditor-3.1

[ckeditor.git] / _source / plugins / table / dialogs / table.js

diff --git a/_source/plugins/table/dialogs/table.js b/_source/plugins/table/dialogs/table.js
index 461e36e..215a373 100644 (file)
--- a/_source/plugins/table/dialogs/table.js
+++ b/_source/plugins/table/dialogs/table.js
@@ -1,5 +1,5 @@
 /*\r
-Copyright (c) 2003-2009, CKSource - Frederico Knabben. All rights reserved.\r
+Copyright (c) 2003-2010, CKSource - Frederico Knabben. All rights reserved.\r
 For licensing, see LICENSE.html or http://ckeditor.com/license\r
 */\r
 \r
@@ -68,6 +68,12 @@ For licensing, see LICENSE.html or http://ckeditor.com/license
                        },\r
                        onOk : function()\r
                        {\r
+                               if ( this._.selectedElement )\r
+                               {\r
+                                       var selection = editor.getSelection(),\r
+                                               bms = editor.getSelection().createBookmarks();\r
+                               }\r
+\r
                                var table = this._.selectedElement || makeElement( 'table' ),\r
                                        me = this,\r
                                        data = {};\r
@@ -174,22 +180,28 @@ For licensing, see LICENSE.html or http://ckeditor.com/license
                                        // Set the width and height.\r
                                        var styles = [];\r
                                        if ( info.txtHeight )\r
-                                               styles.push( 'height:' + info.txtHeight + 'px' );\r
+                                               table.setStyle( 'height', CKEDITOR.tools.cssLength( info.txtHeight ) );\r
+                                       else\r
+                                               table.removeStyle( 'height' );\r
+\r
                                        if ( info.txtWidth )\r
                                        {\r
                                                var type = info.cmbWidthType || 'pixels';\r
-                                               styles.push( 'width:' + info.txtWidth + ( type == 'pixels' ? 'px' : '%' ) );\r
+                                               table.setStyle( 'width', info.txtWidth + ( type == 'pixels' ? 'px' : '%' ) );\r
                                        }\r
-                                       styles = styles.join( ';' );\r
-                                       if ( styles )\r
-                                               table.$.style.cssText = styles;\r
                                        else\r
+                                               table.removeStyle( 'width' );\r
+\r
+                                       if( !table.getAttribute( 'style' ) )\r
                                                table.removeAttribute( 'style' );\r
                                }\r
 \r
                                // Insert the table element if we're creating one.\r
                                if ( !this._.selectedElement )\r
                                        editor.insertElement( table );\r
+                               // Properly restore the selection inside table. (#4822)\r
+                               else\r
+                                       selection.selectBookmarks( bms );\r
 \r
                                return true;\r
                        },\r