From 5301fbf706e73515745206d52594091113bb8f95 Mon Sep 17 00:00:00 2001
From: Joakim <joakim@jnss.me>
Date: Thu, 19 Feb 2026 16:35:18 +0100
Subject: [PATCH] fix: use onMount return for cleanup instead of onDestroy

onDestroy runs during SSR in Svelte 5, causing document reference
errors. The onMount return function only runs on client teardown.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 opal-web/src/lib/components/BottomSheet.svelte | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/opal-web/src/lib/components/BottomSheet.svelte b/opal-web/src/lib/components/BottomSheet.svelte
index e419d20..acbe6f4 100644
--- a/opal-web/src/lib/components/BottomSheet.svelte
+++ b/opal-web/src/lib/components/BottomSheet.svelte
@@ -1,5 +1,5 @@
 <script>
-	import { onMount, onDestroy, afterUpdate } from 'svelte';
+	import { onMount, afterUpdate } from 'svelte';
 
 	/** @type {boolean} */
 	export let open = false;
@@ -72,11 +72,11 @@
 	onMount(() => {
 		mounted = true;
 		document.addEventListener('keydown', handleKeydown);
-	});
 
-	onDestroy(() => {
-		document.removeEventListener('keydown', handleKeydown);
-		document.documentElement.style.overflow = '';
+		return () => {
+			document.removeEventListener('keydown', handleKeydown);
+			document.documentElement.style.overflow = '';
+		};
 	});
 
 	/**