Ticket #3372: 3372_5.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.
@@ -167 +167 @@
         function UndoManager( editor )
         {
                 this.typesCount = 0;
+                this.modifiersCount = 0;
 
                 this.editor = editor;
 
@@ -182 +183 @@
                 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(),
+                                isSelected = !this.editor.getSelection().getRanges()[0].collapsed,
+
+                                // Backspace, Delete
+                                modifierCodes = { 8:1, 46:1 },
+                                isModifier = keystroke in modifierCodes,
+                                wasModifier = this.lastKeystroke in modifierCodes,
+                                lastWasSameModifier = isModifier && keystroke == this.lastKeystroke,
+
+                                // Arrows: L, T, R, B
+                                resetTypingCodes = { 37:1, 38:1, 39:1, 40:1 },
+                                isReset = keystroke in resetTypingCodes,
+                                wasReset = this.lastKeystroke in resetTypingCodes,
+
+                                isTyping = !isModifier && !isReset,
+
+                                modifierSnapshot = ( isModifier && !lastWasSameModifier ) || ( isTyping && isSelected ),
+                                startedTyping = isTyping && ( wasModifier || wasReset );
+
+                        if ( startedTyping || modifierSnapshot )
                         {
                                 var beforeTypeImage = new Image( this.editor );
 
@@ -214 +243 @@
                                                         this.hasRedo = false;
 
                                                         this.typesCount = 1;
-                                                        this.typing = true;
+                                                        this.modifiersCount = 1;
 
                                                         this.onChange();
                                                 }
                                         },
-                                        0, this );
-
-                                return;
+                                        0, this
+                                );
                         }
+                        this.lastKeystroke = keystroke;
 
-                        this.typesCount++;
+                        if ( isModifier )
+                        {
+                                this.typesCount = 0;
+                                this.modifiersCount++;
+
+                                if ( this.modifiersCount > 25 )
+                                {
+                                        this.save();
+                                        this.modifiersCount = 1;
+                                }
+                        }
+                        else if ( !isReset )
+                        {
+                                this.modifiersCount = 0;
+                                this.typesCount++;
 
-                        if ( this.typesCount > 25 )
-                        {
-                                this.save();
-                                this.typesCount = 1;
-                        }
-
-                        this.typing = true;
+                                if ( this.typesCount > 25 )
+                                {
+                                        this.save();
+                                        this.typesCount = 1;
+                                }
+                        }
                 },
 
                 fireChange : function()