Undo manager does not record object resizing

[Ron]

Undo works like a charm, but when resizing an image this cannot be undone with the undo system. It would be great if this would be incorporated.

---

See ​http://dev.ckeditor.com/ticket/1511#comment:20 for reasons why this issue has not been fixed.

[Artur Formella]

Similar bug in V3- In most cases resizing can be undone but undo snapshot isn't made.

[Frederico Caldeira Knabben]

Milestone CKEditor 3.x deleted

[Jakub Ś]

<p>

Type, paste something or start formatting this sample text to see plugin in action. <img alt="" src="​http://upload.wikimedia.org/wikipedia/commons/1/17/Ferrari_F430_front_20080605.jpg" style="width: 225px; height: 160px;" /></p>

This is still an issue in latest CKEditor 4.1. Problem can be reproduced in both CKE 3.x and 4.x (v4) in IE and Firefox and concerns resizing with native browser handles (Please also note that in CKEditor resizing image with CKEditor dialog is registered by undo manager).

From CKEditor 3.0 through 3.6.6 till 4.0 beta you needed to “trigger” undo manager to get this working. If you had used below initial code you had to first e.g. type letter then resize image so that undo manager could catch it. Another option to trigger undo manager was pasting below code in source mode, switching to WYSIWYG and then resizing image

<p>
	Type, paste something or start formatting this sample text to see plugin in action.
	<img alt="" src="http://upload.wikimedia.org/wikipedia/commons/1/17/Ferrari_F430_front_20080605.jpg" style="width: 225px; height: 160px;" /></p>

In CKEditor 4.0 behaviour has changed as resizing image with native handles isn’t caught at all or almost at all. You can still trigger undo manager in the same way as for CKE 3.0 – 4 beta but now image resize won’t be reverted separately. Let’s say you have loaded initial code, typed letter and then resized image. From CKEditor 4.0 pressing undo will revert both typed letter and resized image.

[Alfonso Martínez de Lizarrondo]

I've tested my onchange plugin with Firefox and it's able to detect such native changes, so I guess that when you release the new onchange plugin this feature will be incorporated by default.

[Jakub Ś]

Thanks for comment @alfonsoml. It seems that i haven't checked this properly the first time. New modified comment:5 explains better what is happening.

[David Chandler]

Patch to allow undo of resize in Internet Explorer

[David Chandler]

Added a patch for CKEditor 4.3.3 to allow undo of resizing in Internet Explorer.

[Jakub Ś]

To get your proposed fix tested and applied (if it is correct) please get CKEditor 4 code from ​https://github.com/ckeditor/ckeditor-dev, prepare your fix and make a "pull request".
In comment please specify link to bug for which this fix was prepared.
Here is the example list of pull requests ​https://github.com/ckeditor/ckeditor-dev/pulls?direction=desc&page=1&sort=created&state=open.

This is much faster way to get proposed fix implemented/rejected (if they are invalid). At least in V4.

[Piotrek Koszuliński]

Scheduled for review in 4.4.1. Pull request: ​https://github.com/ckeditor/ckeditor-dev/pull/88

[Piotrek Koszuliński]

I guess that it won't be possible to write reasonable tests for this, so, after creating a ticket branch, do a manual testing and merge it to master if it passes.

[Joel]

Add CC

[Marek Lewandowski]

PR#88 needs some tweaks mentioned in its ​GitHub discussion.

[Piotrek Koszuliński]

During review I encountered IE9+ bug which makes the approach used in pull request invalid (would work only on IE8-10 so it would be a temporary fix). I also found another ticket that is very related to this one - #9317. More details: ​https://github.com/ckeditor/ckeditor-dev/pull/88#issuecomment-42294681

We'll work on both of them in CKEditor 4.4.2. They need a similar code, so we can merge both patches. The patch should be based on miken's workaround from ​https://dev.ckeditor.com/ticket/9317#comment:4

[Piotrek Koszuliński]

Did not make it to the 4.4.2. Related to #11739 that needed to be postponed too (​http://dev.ckeditor.com/ticket/11739#comment:17).

[Piotrek Koszuliński]

The 4.4.3 milestone was shortened, so couple of tickets must be postponed.

[Piotrek Koszuliński]

While working on #9317 we found that there's no possibility to intercept resizing on IE11 and it's very likely that it won't be possible in next IE versions. However, we managed to convince Microsoft that there must be a way to disable native object resize handles (​https://connect.microsoft.com/IE/feedback/details/907422/ie11-unable-to-disable-resize-helper-for-images-inputs-etc-inside-contenteditable). So we are able to handle only old IE versions and not without complications.

A for Firefox, we thought that using the input event we'll be able to handle resizing (there are no special events), but again... no. Input event does not carry any information about intention, so we only know that something changed, but it could be everything and thus we can't automatically record a snapshot, because it might be a keypress (we group keypresses). Of course we could use mutation observers in this case, but that's again a lot of work (mainly testing). Fortunately, ​config.disableObjectResizing works on FF.

Taking all this into consideration, we are not willing to spend time on this issue. Perhaps it can be fixed on all browsers, but cost exceeds profit at the moment. We really hope that Microsoft will implement the 'enableObjectResizing' command (or some alternative) and we'll be able to hide these problematic handles on all browsers.

[Jakub Ś]

#14592 was marked as duplicate.