Fix unload event used in core Javascript (#2812)

* Switching to use pagehide vs deprecated unload event.
gocodebox · Dec 17, 2024 · 21f0ce2 · 21f0ce2
1 parent 9e528d0
commit 21f0ce2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.changelogs/fix_unload-event-in-tracking.yml b/.changelogs/fix_unload-event-in-tracking.yml
@@ -0,0 +1,5 @@
+significance: patch
+type: fixed
+links:
+  - "#2620"
+entry: Fixes Pagespeed notice regarding deprecated Javascript event usage.
diff --git a/assets/js/app/llms-tracking.js b/assets/js/app/llms-tracking.js
@@ -35,7 +35,7 @@ LLMS.Tracking = function( settings ) {
 		self.addEvent( 'page.load' );
 
 		window.addEventListener( 'beforeunload', onBeforeUnload );
-		window.addEventListener( 'unload', onUnload );
+		window.addEventListener( 'pagehide', onUnload );
 
 		document.addEventListener( 'visibilitychange', onVisibilityChange );