-
-
Notifications
You must be signed in to change notification settings - Fork 348
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(edgeless): note does not update when other peers modified note #4808
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
Is it the content or the position that can't be synchronized? In render, there is root's render method, so theoretically, there should be event listeners. |
Both, like when you drag the note or press enter to add a new paragraph. In the previous version (before I wonder if there's a better approach to solving this issue. |
I think we should merge this pr. Although there may have better solutions, we can improve the implementation later. Anyway, it's a fatal bug. And I think we should add some e2e test for this case. |
Got it. I'll merge after pushing some related fixes. |
close #4697
The
BlockSuiteRoot
component has built a rerender mechanism inside. But, it does not work for edgeless mode cause edgeless wraps everything insideSurfaceBlock
which stops the notes that are deep inside the components tree from updating.I added an extra slot listener to track notes' updates individually.