From edd79857522e151a3272b40658fddb411558d895 Mon Sep 17 00:00:00 2001
From: Jules <jules@example.com>
Date: Wed, 15 Jul 2026 16:58:21 +0000
Subject: [PATCH 1/2] Fix scroll-reveal action to observe dynamically added
 elements and remove manual workaround

---
 src/lib/actions/reveal.ts  | 24 ++++++++++++++++++++----
 src/routes/cv/+page.svelte |  9 +--------
 2 files changed, 21 insertions(+), 12 deletions(-)

diff --git a/src/lib/actions/reveal.ts b/src/lib/actions/reveal.ts
index 94a8025..d50ec74 100644
--- a/src/lib/actions/reveal.ts
+++ b/src/lib/actions/reveal.ts
@@ -1,7 +1,12 @@
 export function reveal(
 	node: HTMLElement,
-	options = { threshold: 0.08, rootMargin: '0px 0px -30px 0px' }
+	options: any = { threshold: 0.08, rootMargin: '0px 0px -30px 0px' }
 ) {
+	const observerOptions = {
+		threshold: options?.threshold !== undefined ? options.threshold : 0.08,
+		rootMargin: options?.rootMargin || '0px 0px -30px 0px'
+	};
+
 	const observer = new IntersectionObserver((entries) => {
 		for (const entry of entries) {
 			if (entry.isIntersecting) {
@@ -9,12 +14,23 @@ export function reveal(
 				observer.unobserve(entry.target);
 			}
 		}
-	}, options);
+	}, observerOptions);
+
+	const observeElements = () => {
+		const reveals = node.querySelectorAll('.reveal');
+		reveals.forEach((el) => {
+			if (!el.classList.contains('visible')) {
+				observer.observe(el);
+			}
+		});
+	};
 
-	const reveals = node.querySelectorAll('.reveal');
-	reveals.forEach((el) => observer.observe(el));
+	observeElements();
 
 	return {
+		update(newOptions: any) {
+			observeElements();
+		},
 		destroy() {
 			observer.disconnect();
 		}
diff --git a/src/routes/cv/+page.svelte b/src/routes/cv/+page.svelte
index f1ae064..e00d4f2 100644
--- a/src/routes/cv/+page.svelte
+++ b/src/routes/cv/+page.svelte
@@ -46,13 +46,6 @@
 			.save()
 			.then(async () => {
 				showLinks = true;
-				const { tick } = await import('svelte');
-				await tick();
-				if (container) {
-					container.parentElement?.querySelectorAll('.reveal').forEach((el) => {
-						el.classList.add('visible');
-					});
-				}
 			});
 	}
 </script>
@@ -65,7 +58,7 @@
 	/>
 </svelte:head>
 
-<div use:reveal class="flex flex-col gap-6">
+<div use:reveal={{ trigger: showLinks }} class="flex flex-col gap-6">
 	<div bind:this={container} class="flex flex-col {showLinks ? 'gap-6' : 'gap-2 pb-8'}">
 		<!-- Header -->
 		<section class="section-shell {showLinks ? 'reveal' : ''}">

From b79975a98495e72c1970b85062833a2292b56675 Mon Sep 17 00:00:00 2001
From: Jules <jules@example.com>
Date: Wed, 15 Jul 2026 17:04:27 +0000
Subject: [PATCH 2/2] Fix ESLint errors in reveal action

---
 src/lib/actions/reveal.ts | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/lib/actions/reveal.ts b/src/lib/actions/reveal.ts
index d50ec74..f45ed99 100644
--- a/src/lib/actions/reveal.ts
+++ b/src/lib/actions/reveal.ts
@@ -1,6 +1,12 @@
+export interface RevealOptions {
+	threshold?: number;
+	rootMargin?: string;
+	trigger?: unknown;
+}
+
 export function reveal(
 	node: HTMLElement,
-	options: any = { threshold: 0.08, rootMargin: '0px 0px -30px 0px' }
+	options: RevealOptions = { threshold: 0.08, rootMargin: '0px 0px -30px 0px' }
 ) {
 	const observerOptions = {
 		threshold: options?.threshold !== undefined ? options.threshold : 0.08,
@@ -28,7 +34,7 @@ export function reveal(
 	observeElements();
 
 	return {
-		update(newOptions: any) {
+		update() {
 			observeElements();
 		},
 		destroy() {