Opened 6 years ago

Last modified 3 years ago

#10979 confirmed Bug

Undo does not work for resizing elements such as table, image, horizontal line

Reported by: Zoltan Koszegi Owned by:
Priority: Normal Milestone:
Component: General Version: 4.0 Beta
Keywords: oracle Cc:

Description

I tested with IE9 on full featured DEMO page.
STR:
1) Click the table to get square handles.
2) Resize the table to have different dimensions.
Result: undo does not work
3) Click the image to get square handles.
4) Resize the image to have different dimensions.
Result: undo does not work
3) Click the horizontal line on the bottom of the page to get square handles.
4) Resize the horizontal line to have different dimensions.
Result: undo does not work

Change History (3)

comment:1 Changed 6 years ago by Jakub Ś

Status: newconfirmed
Version: 4.2.14.0 Beta

This is the same type of issue as #10977.

This particular problem can be reproduced in IE and FF.

Since browsers should now support native input event it should be possible to catch such resizing. Below code works in FF but there might be problems with IE (https://developer.mozilla.org/en-US/docs/Web/Reference/Events/input) as it doesn't seem to contenteditable elements.

editor.on('instanceReady', function(e){			
	editor.document.getBody().$.addEventListener("input", function() {
		console.log('input');
	}, false);
});

comment:2 Changed 4 years ago by Piotrek Koszuliński

This particular problem can be reproduced in IE and FF.

This issue isn't reproducible on FF any more because since ~4.4.x we listen to the input event in the undo manager. However, IE does not implement this event for contentEditable yet, so the bug still exists there.

comment:3 Changed 3 years ago by Jakub Ś

#14928 was marked as duplicate.

This issue isn't reproducible on FF anymore because since ~4.4.x

As pointed out by @Wizard in #14928, if resizing is the first action it won't work even in FF.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy