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

Removing widget breaks the document list #11352

Closed
LukaszGudel opened this issue Feb 28, 2022 · 3 comments
Closed

Removing widget breaks the document list #11352

LukaszGudel opened this issue Feb 28, 2022 · 3 comments
Assignees
Labels
package:list squad:core Issue to be handled by the Core team. type:bug This issue reports a buggy (incorrect) behavior.

Comments

@LukaszGudel
Copy link
Contributor

📝 Provide detailed reproduction steps (if any)

  1. Switch to ck/10812-document-list-editing.
  2. Open documentlist manual test.
  3. Remove block image from the list item content.

✔️ Expected result

Only the selected image is removed. Other elements inside this list item are still there.

❌ Actual result

The list breaks into 2 lists and requires manual merges to get to the expected result.

cke5-document-list-removing-widget-breaks-list.mp4
@LukaszGudel LukaszGudel added type:bug This issue reports a buggy (incorrect) behavior. package:list squad:core Issue to be handled by the Core team. labels Feb 28, 2022
@CatStrategist
Copy link
Contributor

@LukaszGudel This one should be resolved, please check it out.

@LukaszGudel
Copy link
Contributor Author

@CatStrategist Yes. It's fixed on the current master branch.

@CatStrategist CatStrategist self-assigned this Apr 5, 2022
@CKEditorBot CKEditorBot added this to the upcoming milestone Apr 5, 2022
@CatStrategist
Copy link
Contributor

Closed in #11425.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:list squad:core Issue to be handled by the Core team. type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

No branches or pull requests

4 participants