-
Notifications
You must be signed in to change notification settings - Fork 25
fix version-restoring #830
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
Conversation
6282e0f to
0e7c1b5
Compare
0e7c1b5 to
a3d48f2
Compare
benjaminfrueh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @hweihwang for this fix! I tested this and it worked good for me.
|
Did you also test explicitly on stable29? |
Thanks for the reminder, yes I have now and it works as expected. In stable29 it also correctly shows the readonly version preview and restoring versions works generally too - but I have experienced a potential problem and have one question: When I am connected to a whiteboard with multiple users for collaboration, the version restore seems not to restore anything, maybe because this is synced back from the other connected user immediately then? |
@hweihwang Can you elaborate on that? |
Signed-off-by: Hoang Pham <hoangmaths96@gmail.com>
a3d48f2 to
e315cfb
Compare
|
Thanks @benjaminfrueh, @juliusknorr, that's a bug there, restoring using the right side panel without entering the version preview mode should work. But when entering the preview mode, it just got disconnected from the collab socket, so the restore couldn’t broadcast SCENE_RESTORE and the other users’ newer scenes merged back in and overwrote it. I changed it so we stay connected in preview, please re-test it again, thanks! |
|
Hi @hweihwang - this fixes the version restoring for me also during collaboration. There is another issue I experienced when collaborating: If User A restores a whiteboard, then the canvas of User B gets resized to the size of User A's canvas. This is more obvious if one user restores with a smaller resolution or smaller window size. Maybe this could be improved in a different Issue/PR. |
No description provided.