From 3b702a13997d621ea339c03dc85eacc27b90ab3e Mon Sep 17 00:00:00 2001
From: Nathan Cummins <nathan@nathancummins.com.au>
Date: Thu, 15 Oct 2026 12:19:04 +1030
Subject: [PATCH] Prevent duplicate form and navbar listeners after page
 transitions, tidy env, env example

---
 .env.example                     |  7 +++++++
 src/components/ContactForm.astro |  3 ++-
 src/components/Navbar.astro      | 29 ++++++++++++++++++-----------
 src/env.d.ts                     |  2 +-
 4 files changed, 28 insertions(+), 13 deletions(-)
 create mode 100644 .env.example

diff --git a/.env.example b/.env.example
new file mode 100644
index 0000000..284b616
--- /dev/null
+++ b/.env.example
@@ -0,0 +1,7 @@
+# Copy to `.env` and fill in real values. Never commit `.env`.
+EMAIL_HOST=smtp.example.com
+EMAIL_PORT=587
+EMAIL_SECURE=false
+EMAIL_USER=user@example.com
+EMAIL_PASS=change-me
+EMAIL_FROM=noreply@example.com
diff --git a/src/components/ContactForm.astro b/src/components/ContactForm.astro
index ff6741a..869b126 100644
--- a/src/components/ContactForm.astro
+++ b/src/components/ContactForm.astro
@@ -77,7 +77,8 @@ const { id: ID = "contact-form" } = Astro.props;
 <script define:vars={{ ID }}>
   function init() {
     const form = document.getElementById(ID);
-    if (!form) return;
+    if (!form || form.dataset.initialised) return;
+    form.dataset.initialised = "true";
     const responseMsg = form.querySelector(
       `#${CSS.escape(ID + "-response-message")}`
     );
diff --git a/src/components/Navbar.astro b/src/components/Navbar.astro
index 306deaf..dd094be 100644
--- a/src/components/Navbar.astro
+++ b/src/components/Navbar.astro
@@ -139,7 +139,8 @@ const { navbarDisplay = "normal" } = Astro.props as Props;
     nav = document.getElementById("nav");
     toggle = document.getElementById("menu-toggle");
 
-    if (toggle) {
+    if (toggle && !toggle.dataset.bound) {
+      toggle.dataset.bound = "true";
       toggle.addEventListener("click", () => {
         if (nav) {
           const visible = nav.getAttribute("data-mobile-menu") === "visible";
@@ -151,18 +152,24 @@ const { navbarDisplay = "normal" } = Astro.props as Props;
       });
     }
 
-    document.addEventListener("keydown", (e) => {
-      if (
-        e.key === "Escape" &&
-        nav?.getAttribute("data-mobile-menu") === "visible"
-      ) {
-        nav.setAttribute("data-mobile-menu", "hidden");
-        toggle?.setAttribute("aria-expanded", "false");
-      }
-    });
+    if (!document.documentElement.dataset.navbarKeydownBound) {
+      document.documentElement.dataset.navbarKeydownBound = "true";
+      document.addEventListener("keydown", (e) => {
+        if (
+          e.key === "Escape" &&
+          nav?.getAttribute("data-mobile-menu") === "visible"
+        ) {
+          nav.setAttribute("data-mobile-menu", "hidden");
+          toggle?.setAttribute("aria-expanded", "false");
+        }
+      });
+    }
 
     checkScroll();
   });
 
-  window.addEventListener("scroll", checkScroll);
+  if (!document.documentElement.dataset.navbarScrollBound) {
+    document.documentElement.dataset.navbarScrollBound = "true";
+    window.addEventListener("scroll", checkScroll, { passive: true });
+  }
 </script>
diff --git a/src/env.d.ts b/src/env.d.ts
index 2979382..0c6b0a7 100644
--- a/src/env.d.ts
+++ b/src/env.d.ts
@@ -1,4 +1,4 @@
-/// <reference types="@astrojs/image/client" />
+/// <reference types="astro/client" />
 /// <reference types="./types/SiteImage" />
 
 interface Window {