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

Following logic #307

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Following logic #307

wants to merge 1 commit into from

Conversation

hweihwang
Copy link
Contributor

No description provided.

@hweihwang hweihwang self-assigned this Dec 11, 2024
@hweihwang hweihwang force-pushed the poc-whiteboard-recording-server branch 3 times, most recently from f42593c to 86d2680 Compare December 12, 2024 09:30
@@ -151,6 +152,14 @@ export class Portal {
case BroadcastType.MouseLocation:
this.collab.updateCursor(decoded.payload)
break
case BroadcastType.ViewportUpdate:
this.collab.updateCollaboratorViewport(
decoded.payload.userId,
Copy link
Member

Choose a reason for hiding this comment

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

Just an idea, we could also pass the viewport size to the backend to record depending on the screens size of the user?

@hweihwang hweihwang force-pushed the poc-whiteboard-recording-server branch from 86d2680 to 4837c3c Compare December 17, 2024 08:50
Signed-off-by: Hoang Pham <[email protected]>
@hweihwang hweihwang force-pushed the poc-whiteboard-recording-server branch from 4837c3c to 0c4db45 Compare December 19, 2024 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants