Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

2SCM fails when a link is next to an image (widget) #7437

Closed
pomek opened this issue Jun 16, 2020 · 0 comments · Fixed by #9023
Closed

2SCM fails when a link is next to an image (widget) #7437

pomek opened this issue Jun 16, 2020 · 0 comments · Fixed by #9023
Labels
package:link package:widget squad:magic type:bug This issue reports a buggy (incorrect) behavior.

Comments

@pomek
Copy link
Member

pomek commented Jun 16, 2020

📝 Provide detailed reproduction steps (if any)

  1. yarn run manual -f ckeditor5
  2. http://localhost:8125/ckeditor5/tests/manual/all-features.html
  3. Paste to the console:
editor.setData( '<p><a href="https://cksource.com">https://cksource.com</a></p><figure class="image"><img src="sample.jpg" alt=""></figure>' );
  1. Click at the end of the link and use the keyboard to put the selection after the link.

✔️ Expected result

The selection loses the linkHref attribute and I can type after the link.

image

❌ Actual result

The image is selected.

image


If you'd like to see this fixed sooner, add a 👍 reaction to this post.

@pomek pomek added type:bug This issue reports a buggy (incorrect) behavior. package:link package:widget labels Jun 16, 2020
@pomek pomek changed the title 2SCM fails when a link is next to an image 2SCM fails when a link is next to an image (widget) Jun 16, 2020
@Reinmar Reinmar added this to the nice-to-have milestone Jul 27, 2020
Reinmar added a commit that referenced this issue Mar 8, 2021
Feature (engine): Introduced bubbling of `view.Document` events, similar to how bubbling works in the DOM. Bubbling allows listening on a view event on a specific kind of an element, hence simplifying code that needs to handle a specific event for only that element (e.g. `enter` in `blockquote`s only). Read more in the documentation: **\[TODO\]**. Closes #8640.

Feature (engine): Introduced `ArrowKeysObserver`. See #8640.

Fix (utils): The `EmitterMixin#listenTo()` method is split into listener and emitter parts. The `ObservableMixin` decorated methods reverted to the original method while destroying an observable.

Other (typing): The `TwoStepCaretMovement` feature is now using bubbling events. Closes #7437.

BREAKING CHANGE: We introduced bubbling of `view.Document` events, similar to how bubbling works in the DOM. That allowed us to reprioritize many listeners that previously had to rely on `priority`. However, it means that existing listeners that use priorities may now be executed in a wrong moment. The listeners to such events should be reviewed in terms of when they should be executed (in what context/element/phase). You can find more information regarding bubbling in the documentation: **\[TODO\]**. See #8640.

Internal (widget): The enter, delete and arrow key events handling moved to the usage of the bubbling observer.

Internal (block-quote): The enter and delete events handling moved to the usage of the bubbling observer.

Internal (code-block): The enter event handling moved to the usage of the bubbling observer.

Internal (list): The enter and delete events handling moved to the usage of the bubbling observer.

Internal (table): The arrow keys handling moved to the usage of the bubbling observer.
@Reinmar Reinmar modified the milestones: nice-to-have, iteration 41 Mar 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:link package:widget squad:magic type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants