Changeset 5669

Timestamp:

07/09/10 09:52:23 (3 years ago)

Author:

tobiasz.cudnik

Message:

Fixed #5415 Undo not working when we change the Table Properties for the table on a saved page

Location:

CKEditor/branches/versions/3.4.x

Files:

• CHANGES.html (modified) (1 diff)
• _source/plugins/dialog/plugin.js (modified) (1 diff)
• _source/plugins/div/dialogs/div.js (modified) (2 diffs)

CKEditor/branches/versions/3.4.x/CHANGES.html

@@ -42 +42 @@
                 <li><a href="http://dev.fckeditor.net/ticket/5628">#5628</a> : Port 'DragResizeTable' plugin from FCKEditor 2.x.</li>
                 <li><a href="http://dev.fckeditor.net/ticket/979">#979</a> : New configuration 'enableTabKeyTools' to allow using 'Tab' key to navigate through table cells.</li>
+        </ul>
+        <p>
+                        Fixed issues:</p>
+        <ul>
+                <li><a href="http://dev.fckeditor.net/ticket/5415">#5415</a> : Undo not working when we change the Table Properties for the table on a saved page.</li>
         </ul>
         <h3>

CKEditor/branches/versions/3.4.x/_source/plugins/dialog/plugin.js

@@ -169 +169 @@
                         this.on( 'ok', function( evt )
                                 {
+                                        // Dialog confirm might probably introduce content changes (#5415).
+                                        editor.fire( 'saveSnapshot' );
+                                        setTimeout( function () { editor.fire( 'saveSnapshot' ); }, 0 );
                                         if ( definition.onOk.call( this, evt ) === false )
                                                 evt.data.hide = false;

CKEditor/branches/versions/3.4.x/_source/plugins/div/dialogs/div.js

@@ -485 +485 @@
                         onOk : function()
                         {
-                                editor.fire( 'saveSnapshot' );
                                 if ( command == 'editdiv' )
                                         containers = [ this._element ];
@@ -500 +499 @@
                                         !containers[ i ].getAttribute( 'style' ) && containers[ i ].removeAttribute( 'style' );
                                 }
-                                editor.fire( 'saveSnapshot' );
 
                                 this.hide();