wiris / html-integrations

The official JavaScript library for MathType, the leading formula editor and equation writer for the web by Wiris
https://wiris.com/solutions/integrations/html-editors/
MIT License
76 stars 53 forks source link

[CKEditor 5] CKeditor error: "Cannot read property 'name' of undefined" when trying to edit the formula on a TodoList #12

Closed FilipTokarski closed 4 years ago

FilipTokarski commented 4 years ago

Tested on new version, branch ckeditor5-demo

Steps to reproduce:

Result:

   Uncaught TypeError: Cannot read property 'name' of undefined
    at Mapper.getModelLength (mapper.js:485)
    at Mapper._findPositionIn (mapper.js:550)
    at Mapper.on.priority (mapper.js:107)
    at Mapper.fire (emittermixin.js:209)
    at Mapper.toViewPosition (mapper.js:334)
    at DowncastDispatcher.<anonymous> (downcasthelpers.js:399)
    at DowncastDispatcher.fire (emittermixin.js:209)
    at DowncastDispatcher.convertRemove (downcastdispatcher.js:213)
    at DowncastDispatcher.convertChanges (downcastdispatcher.js:139)
    at editingcontroller.js:92
carla-at-wiris commented 4 years ago

This issue can no longer be reproduced, so we assume it's solved. But we detected that the todo-list plugin has positioning issues:

ck5issue As you can see in the first image, there is an all-list created but the items in the list boxes do not render well, however, if you overlay the later image by aligning the items to the center of the editor, they look good:

issueck52

FilipTokarski commented 4 years ago

Thank you for reporting this. I'm not able to reproduce this positioning issue in any environment, so I suppose it's nothing critical for now, however I'll keep this in mind.

As it comes to the original issue, I'm also unable to reproduce it ( using new version 7.24.1 ). It looks like it's been fixed and we can close this ticket as solved.

carla-at-wiris commented 4 years ago

Thank you!