Ticket #3372: 3372_4.patch

_source/plugins/undo/plugin.js

@@ -82 +82 @@
                                                                 {
                                                                         // Do not capture CTRL hotkeys.
                                                                         if ( !event.data.$.ctrlKey && !event.data.$.metaKey )
-                                                                                undoManager.type();
+                                                                                undoManager.type( event );
                                                                 });
 
                                                         // Being this the first call, let's get an undo snapshot.
@@ -182 +182 @@
                 this.index = -1;
 
                 this.limit = editor.config.undoStackSize;
+
+                /**
+                 * Remember last pressed key.
+                 */
+                this.lastKeystroke = 0;
         }
 
         UndoManager.prototype =
         {
-                type : function()
+                /**
+                 *
+                 * @param {CKEDITOR.dom.event} [event]
+                 */
+                type : function( event )
                 {
-                        if ( !this.typing )
+                        var keystroke = event && event.data.getKeystroke(),
+                                // Backspace, Delete
+                                modifierCodes = { 8:1, 46:1 },
+                                // Arrows: L, T, R, B
+                                resetTypingCodes = { 37:1, 38:1, 39:1, 40:1 },
+                                isModifier = keystroke in modifierCodes,
+                                lastWasSameModifier = isModifier && keystroke == this.lastKeystroke,
+                                modifierSnapshot = isModifier  && !lastWasSameModifier;
+                        
+                        if ( !this.typing || modifierSnapshot )
                         {
                                 var beforeTypeImage = new Image( this.editor );
 
@@ -219 +237 @@
                                                         this.onChange();
                                                 }
                                         },
-                                        0, this );
+                                        0, this
+                                );
 
-                                return;
+                                if ( !modifierSnapshot )
+                                        return;
                         }
+                        this.lastKeystroke = keystroke;
 
-                        this.typesCount++;
+                        if ( keystroke in resetTypingCodes )
+                                this.typing = false;
+                        else
+                        {
+                                this.typesCount++;
 
-                        if ( this.typesCount > 25 )
-                        {
-                                this.save();
-                                this.typesCount = 1;
-                        }
+                                if ( this.typesCount > 25 )
+                                {
+                                        this.save();
+                                        this.typesCount = 1;
+                                }
 
-                        this.typing = true;
+                                this.typing = true;
+                        }
                 },
 
                 fireChange : function()