Opened 4 years ago

Closed 2 years ago

#13425 closed Bug (expired)

Autoembed: breaking the link before embed loads

Reported by: Szymon Cofalik Owned by:
Priority: Normal Milestone:
Component: General Version: 4.5.0
Keywords: Cc:

Description

When you paste autoembedable link into the editor and break it (using enter key) before the embed content loads, broken part is not removed.

Steps to reproduce:

  1. In empty editor, paste https://www.flickr.com/photos/polandmfa/7005560084/.
  2. Before the link is converted, press left key and enter key.
  3. Wait for embed content to load.

Expected result: The editor contains only embeded content.
Result: The editor contains embeded content and leftovers from the link.

Change History (5)

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

Keywords: chrome ie ff opera autoembed removed
Status: newpending

Not a bug IMO. User intentionally split this link, so why would be the rest of it removed? I think that we either need to cancel embedding or just keep doing what we do.

Let's discuss this.

comment:2 Changed 4 years ago by Szymon Cofalik

I reported this because it was specified as one of steps in manual test. "Breaking the link in two before it's embedded." Unless it meant breaking with <br /> or space.

http://tests.ckeditor.dev:10450/tests/plugins/autoembed/manual/autoembed

Last edited 4 years ago by Szymon Cofalik (previous) (diff)

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

Right... The instruction isn't clear - this is the expected behaviour when breaking. Question is - is it expected by me only or is it generally expected :D

comment:4 Changed 4 years ago by Szymon Cofalik

Keywords: opera ie ff chrome autoembed added

comment:5 Changed 2 years ago by Jakub Ś

Keywords: opera ie ff chrome autoembed removed
Resolution: expired
Status: pendingclosed
Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy