-
Notifications
You must be signed in to change notification settings - Fork 388
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
fix(pte): cursor presence causes pte error #6622
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
No changes to documentation |
Component Testing Report Updated May 13, 2024 4:23 PM (UTC)
|
Since the leaf is now properly rendered for the presence decorator, we are interested in the next nextSibling
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great catch @pedrobonamin, thank you for this!
I've updated the test to reflect the new DOM structure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work @pedrobonamin!
* fix(pte): cursor presence causes pte error * test(playwright-ct): fix outdated test Since the leaf is now properly rendered for the presence decorator, we are interested in the next nextSibling * fix(pte): update tests --------- Co-authored-by: Per-Kristian Nordnes <per.kristian.nordnes@gmail.com>
Description
Structure consistently crashes with
Cannot resolve a DOM point from Slate
point when editing a document with multiple usersI've been debugging this and it seems to be generated because the
PresenceCursor
element is not rendering theLeaf
that is generated in the editable component, which causes that the editable leaf "dissapears" and crashes the editor.This were the original steps to reproduce the issue:
Steps to reproduce
With the changes introduced I was not able to reproduce the issue anymore,
User 1's structure will then crash
What to review
Testing
Notes for release