Opened 4 years ago

Last modified 4 years ago

#13413 confirmed Bug

[Autoembed] Multiple embedding actions are registered in a single undo step.

Reported by: Olek Nowodziński Owned by:
Priority: Normal Milestone:
Component: UI : Widgets Version: 4.5.0
Keywords: Cc:

Description (last modified by Olek Nowodziński)

  1. Visit http://tests.ckeditor.dev:1030/tests/plugins/autoembed/manual/autoembed
  2. Copy embeddable URL, i.e. https://twitter.com/reinmarpl/status/573118615274315776.
  3. Focus editable.
  4. Hold CTRL+V for a while.
  5. A number of links is pasted and corresponding embed requests turn them into widgets.
  6. CTRL+Z.

Expected: The last embeddable object invoked is reverted back to link.

Actual: All widgets are reverted back to the chain of links. Sometimes only some group of widgets is reverted. The point is the invocation of multiple widgets is registered within a single undo step.

Change History (4)

comment:1 Changed 4 years ago by Olek Nowodziński

Description: modified (diff)

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

Milestone: CKEditor 4.5.1
Status: newconfirmed

Absolute edge case ;) Holding CTRL+V or pasting very quickly will cause problems and that isn't a surprise or something that I would be worried.

comment:3 in reply to:  2 Changed 4 years ago by Olek Nowodziński

Replying to Reinmar:

Absolute edge case ;) Holding CTRL+V or pasting very quickly will cause problems and that isn't a surprise or something that I would be worried.

My point was that it might be just a tip of an iceberg, something serious in undo.

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

Yes and no :D. Depends on whether pastebin is used or not. And autoembed is a special case, because it is asynchronus, so all that multiplies. Ir would be very hard to record all that properly.

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