Is beforeunload a correct event for reseting awareness state?
mifopen opened this issue · 3 comments
mifopen commented
Shouldn't it be just unload
. beforeunload
is often used to ask users about unsaved changes. It means that the user will be "disconnected" from the awareness protocol until they choose, for example, "exit without saving", "cancel" or "save".
dmonad commented
Can you try out whether disconnecting will still work if we do it in the unload event?
mifopen commented
Yes, it does work
dmonad commented
Thanks for verifying. Added this change and made a release.