Skip to content
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 cursors demo issues #179

Merged
merged 2 commits into from
Sep 27, 2023
Merged

Fix cursors demo issues #179

merged 2 commits into from
Sep 27, 2023

Conversation

dpiatek
Copy link
Contributor

@dpiatek dpiatek commented Sep 21, 2023

  • fixes cursors.set throwing an error because we call it before a space was entered
  • fixes cursors not being removed after a user has left

@dpiatek dpiatek force-pushed the fix-cursors-demo-issues branch 2 times, most recently from 31688aa to b5ad1ad Compare September 25, 2023 09:56
Copy link
Collaborator

@lawrence-forooghian lawrence-forooghian left a comment

Choose a reason for hiding this comment

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

I have taken a look at this but am not 100% sure about the changes to the Cursors component given my relative unfamiliarity with the product.

demo/src/components/CurrentSlide.tsx Show resolved Hide resolved
demo/src/components/Cursors.tsx Show resolved Hide resolved
demo/src/components/Cursors.tsx Outdated Show resolved Hide resolved
Dominik Piatek added 2 commits September 27, 2023 08:57
This simplifies the cursors component and fixes the issue where cursor icons would stay even after a member has left.
@dpiatek dpiatek merged commit 99d8200 into main Sep 27, 2023
5 checks passed
@dpiatek dpiatek deleted the fix-cursors-demo-issues branch September 27, 2023 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants