Skip to content

[lexical-react] Fix: update cursor positions on awareness update in collab v1#7960

Merged
etrepum merged 1 commit intofacebook:mainfrom
james-atticus:collab-v1-cursor-syncing
Oct 31, 2025
Merged

[lexical-react] Fix: update cursor positions on awareness update in collab v1#7960
etrepum merged 1 commit intofacebook:mainfrom
james-atticus:collab-v1-cursor-syncing

Conversation

@james-atticus
Copy link
Copy Markdown
Contributor

@james-atticus james-atticus commented Oct 30, 2025

Description

During the refactoring of useYjsCollaboration done as part of the collab v2 implementation, I inadvertently dropped the awareness.on('update',...) call. This brings it back.

Closes #7959

Test plan

collab.v1.cursor.sync.mov

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 30, 2025
@vercel
Copy link
Copy Markdown

vercel bot commented Oct 30, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
lexical Ready Ready Preview Comment Oct 30, 2025 11:23pm
lexical-playground Ready Ready Preview Comment Oct 30, 2025 11:23pm

@etrepum etrepum added this pull request to the merge queue Oct 31, 2025
Merged via the queue into facebook:main with commit 174580d Oct 31, 2025
43 checks passed
@etrepum etrepum mentioned this pull request Oct 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. extended-tests Run extended e2e tests on a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Collaboration selection and cursor position not synced properly in v0.38.1

2 participants