From d3f997405cafe850b7fd86cefeef7ecffe0c2757 Mon Sep 17 00:00:00 2001
From: Sebastian Martinez <me@sebastinez.dev>
Date: Tue, 3 Sep 2024 18:13:05 +0200
Subject: [PATCH] Remove event dispatcher and make required props explicit

Svelte has deprecated the event dispatcher functionality and instead
works with callback functions that are called when an event should in
the past have been called.

Also `export let var: string` is in svelte something that can't be
optional, so we should type it out as `export let var = string |
undefined = undefined` or make sure we always pass that prop.
---
 src/components/Header.svelte |  2 +-
 src/components/Link.svelte   | 12 +++++-------
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/src/components/Header.svelte b/src/components/Header.svelte
index 2a7ffc5..0960232 100644
--- a/src/components/Header.svelte
+++ b/src/components/Header.svelte
@@ -6,7 +6,7 @@
   import Popover from "./Popover.svelte";
   import ThemeSwitch from "./ThemeSwitch.svelte";
 
-  export let currentPage: string;
+  export let currentPage: string | undefined = undefined;
 </script>
 
 <style>
diff --git a/src/components/Link.svelte b/src/components/Link.svelte
index a751fd0..070a035 100644
--- a/src/components/Link.svelte
+++ b/src/components/Link.svelte
@@ -1,15 +1,11 @@
-<script lang="ts" strictEvents>
+<script lang="ts">
   import type { Route } from "@app/lib/router/definitions";
 
-  import { createEventDispatcher } from "svelte";
   import { push, routeToPath } from "@app/lib/router";
 
   export let route: Route;
   export let disabled: boolean = false;
-
-  const dispatch = createEventDispatcher<{
-    afterNavigate: null;
-  }>();
+  export let afterNavigate: (() => void) | undefined = undefined;
 
   function navigateToRoute(event: MouseEvent): void {
     event.preventDefault();
@@ -18,7 +14,9 @@
     }
 
     void push(route);
-    dispatch("afterNavigate");
+    if (afterNavigate) {
+      afterNavigate();
+    }
   }
 </script>