fix: Adding a pagehide listener to cover bases when unload becomes de…

…precated (#298)
adaptlearning · Nov 15, 2023 · 177de84 · 177de84
1 parent f07a603
commit 177de84
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/js/adapt-stateful-session.js b/js/adapt-stateful-session.js
@@ -114,7 +114,7 @@ export default class StatefulSession extends Backbone.Controller {
     if (shouldCommitOnVisibilityChange) {
       document.addEventListener('visibilitychange', this.onVisibilityChange);
     }
-    $(window).on('beforeunload unload', this.endSession);
+    $(window).on('beforeunload unload pagehide', this.endSession);
   }
 
   async saveSessionState() {
@@ -261,7 +261,7 @@ export default class StatefulSession extends Backbone.Controller {
   }
 
   removeEventListeners() {
-    $(window).off('beforeunload unload', this.endSession);
+    $(window).off('beforeunload unload pagehide', this.endSession);
     document.removeEventListener('visibilitychange', this.onVisibilityChange);
     this.stopListening();
   }

diff --git a/required/log_output.html b/required/log_output.html
@@ -52,7 +52,7 @@
     </script>
   </head>
 
-  <body onload="onLoad();" onunload="onUnload();">
+  <body onload="onLoad();" onunload="onUnload();" onpagehide="onUnload();">
     <div id="logDiv"></div>
   </body>