Skip to content
This repository has been archived by the owner on May 2, 2024. It is now read-only.

Add multiplayer cursors and cursor chat #89

Merged
merged 2 commits into from
Feb 18, 2024
Merged

Add multiplayer cursors and cursor chat #89

merged 2 commits into from
Feb 18, 2024

Conversation

reesericci
Copy link
Member

@reesericci reesericci commented Feb 14, 2024

This PR adds multiplayer cursors and cursor chat support through the use of Y.js, action cable, and https://github.com/obl-ong/cursor-chat-actioncable (forked from jackyzha0/cursor-chat)

This only adds multiplayer cursors to anything using the domain layout, that way it's scoped to a domain, and doesn't appear on every page. This also doesn't add synced text input, so the experience isn't seamless.

This comment was marked as off-topic.

@reesericci
Copy link
Member Author

Related to #29

@ajs256
Copy link
Member

ajs256 commented Feb 14, 2024

What's the point of this without shared domains?

@reesericci
Copy link
Member Author

If you share an account through say a team@ email or otherwise, it also lays the groundwork for full shared domains in 2.0

Copy link
Member

@ajs256 ajs256 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks alright, haven't tested

@reesericci
Copy link
Member Author

reesericci commented Feb 16, 2024

I think we should test something like this, has the potential to cause chaos if I messed up the scoping or smthn

Copy link
Collaborator

@polypixeldev polypixeldev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@reesericci reesericci mentioned this pull request Feb 18, 2024
@reesericci reesericci merged commit 6de6737 into main Feb 18, 2024
6 checks passed
@reesericci reesericci deleted the cursor-chat branch February 18, 2024 02:55
@reesericci reesericci added this to the v1.4 milestone Feb 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants