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

Deprecation of unload event in google chrome #3481

Closed
oliverfoster opened this issue Nov 1, 2023 · 2 comments · Fixed by adaptlearning/adapt-contrib-spoor#298 or adaptlearning/adapt-contrib-xapi#125
Labels

Comments

@oliverfoster
Copy link
Member

oliverfoster commented Nov 1, 2023

Subject of the issue

Lars Halkjær from the community forum brings this up on thread https://community.adaptlearning.org/mod/forum/discuss.php?d=4203#p15459

Reference article: https://developer.chrome.com/blog/deprecating-unload/

@oliverfoster
Copy link
Member Author

oliverfoster commented Nov 1, 2023

All of the above event listeners are for both beforeunload and unload, it may be wise to add pagehide as well.

SCORM is a non-restorable connection, such that if a user sends LMS_Finish when leaving the page, re-entering the page through browser navigate cannot alone reinitialise the SCORM session. I don't think we need to worry about the bfcache (back/forward browser history cache) as a result.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment