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

Occasionally error when Move range between different roots. #5601

Closed
misterPeng opened this issue Oct 16, 2019 · 1 comment
Closed

Occasionally error when Move range between different roots. #5601

misterPeng opened this issue Oct 16, 2019 · 1 comment
Labels
resolution:duplicate This issue is a duplicate of another issue and was merged into it. type:bug This issue reports a buggy (incorrect) behavior.

Comments

@misterPeng
Copy link

misterPeng commented Oct 16, 2019

📝 Provide detailed reproduction steps (if any)

  1. On MultirootEditor.I listened to the view render event to determine if the height of the dom of root where the selection is located exceeds maxHeight. If so, then root exceeds maxHeight's range move to nextRoot.mul-editor.zip is my code.
  2. Enter a lot of content, so that it can hold more than 3 roots.
  3. Enter the content in the first root, triggering the operation described in the first point.There will be an accidental bug.

✔️ Expected result

The operation described in the first point is no bug.

❌ Actual result

remove.js?802d:17 Uncaught TypeError: Cannot read property 'parentNode' of undefined

at remove (remove.js?802d:17)
at Renderer._updateChildren (renderer.js?fb19:571)
at Renderer.render (renderer.js?fb19:203)
at View._render (view.js?4b17:676)
at View.eval (view.js?4b17:187)
at View.fire (emittermixin.js?da0d:207)
at View.change (view.js?4b17:478)
at View._disableRendering (view.js?4b17:663)
at Model.EditingController.listenTo.priority (editingcontroller.js?d38d:82)
at Model.fire (emittermixin.js?da0d:207)`

at Renderer._updateChildren (renderer.js?fb19:571), the i == actualDomChildren.length.Is it caused by my code, because the api call error?

📃 Other details

mul-editor.zip

  • Browser: chrome 77.0.3865.120
  • CKEditor version: 12.4.0

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

@Reinmar
Copy link
Member

Reinmar commented Jan 21, 2020

It seems that it's related to couple other issues that we've seen. Closing for now as DUP of #6092.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
resolution:duplicate This issue is a duplicate of another issue and was merged into it. type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

No branches or pull requests

2 participants