perf: prevent running withoutTransition on initial load (#136)

Author: huntabyte

.changeset/eighty-snakes-relax.md

@@ -0,0 +1,5 @@
+---
+"mode-watcher": patch
+---
+
+perf: prevent running `withoutTransition` on initial load as there is nothing to transition from

packages/mode-watcher/src/lib/without-transition.ts

@@ -2,11 +2,23 @@
 
 let timeoutAction: number;
 let timeoutEnable: number;
+/**
+ * Whether this is the first time the function has been
+ * called, which will be true for the initial load, where
+ * we shouldn't need to disable any transitions, as there
+ * is nothing to transition from.
+ */
+let hasLoaded = false;
 
 // Perform a task without any css transitions
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function withoutTransition(action: () => any) {
 	if (typeof document === "undefined") return;
+	if (!hasLoaded) {
+		hasLoaded = true;
+		action();
+		return;
+	}
 	// Clear fallback timeouts
 	clearTimeout(timeoutAction);
 	clearTimeout(timeoutEnable);

packages/mode-watcher/src/routes/+layout.svelte

@@ -5,5 +5,5 @@
 	let { children } = $props();
 </script>
 
-<ModeWatcher themeColors={{ dark: "black", light: "white" }} />
+<ModeWatcher themeColors={{ dark: "black", light: "white" }} disableTransitions />
 {@render children()}