refactor: Simplify cleanup and event listener management in layout

commit: ca9d24adb255378e3c25780d157164e7f204541a
author: Kit · 20mo ago · 2025-02-14 16:52 UTC
parent: 67ec9e6
signature: SSH UnverifiedSHA256:H91DFuEG4jpk5edzLA4aUKkpv4zv9Q339ipRmkph6Tk The key is unknown

1 file · +0 -11

M src/routes/+layout.svelte +0 -11


@@ -93,23 +93,12 @@
93	93		// Add visibility change listener
94	94		document.addEventListener('visibilitychange', handleVisibilityChange);
95	95
96		-	// Add beforeunload listener
97		-	window.addEventListener('beforeunload', cleanup);
98		-
99	96		// Cleanup on unmount
100	97		return () => {
101	98		window.removeEventListener('beforeunload', cleanup);
102		-	cleanup();
103	99		};
104	100		});
105	101
106		-	// Additional safety net for clean navigation within the app
107		-	onDestroy(() => {
108		-	if (!browser) return;
109		-	console.debug('Component being destroyed - Running cleanup');
110		-	window.removeEventListener('beforeunload', cleanup);
111		-	cleanup();
112		-	});
113	102
114	103		$effect(() => {
115	104		if (!browser) return;


@@ -93,23 +93,12 @@
93		// Add visibility change listener	93	// Add visibility change listener
94		document.addEventListener('visibilitychange', handleVisibilityChange);	94	document.addEventListener('visibilitychange', handleVisibilityChange);
95			95
96	-	// Add beforeunload listener
97	-	window.addEventListener('beforeunload', cleanup);
98	-
99		// Cleanup on unmount	96	// Cleanup on unmount
100		return () => {	97	return () => {
101		window.removeEventListener('beforeunload', cleanup);	98	window.removeEventListener('beforeunload', cleanup);
102	-	cleanup();
103		};	99	};
104		});	100	});
105			101
106	-	// Additional safety net for clean navigation within the app
107	-	onDestroy(() => {
108	-	if (!browser) return;
109	-	console.debug('Component being destroyed - Running cleanup');
110	-	window.removeEventListener('beforeunload', cleanup);
111	-	cleanup();
112	-	});
113			102
114		$effect(() => {	103	$effect(() => {
115		if (!browser) return;	104	if (!browser) return;