Minify JS and resolve accessibility issues (back to 100% in Lighthouse)

src/lib/components/Navigation.svelte

@@ -14,20 +14,27 @@
         <span class="mobile nav-header-logo">
             <Wordmark />
         </span>
-        <label
+        <button
-            id="nav-toggle-label"
+            type="button"
-            for="nav-toggle"
+            id="nav-toggle-button"
-            class="nav-toggle"
+            class="btn-clear"
             aria-controls="nav-menu"
             aria-label="Toggle navigation"
             aria-expanded="false"
         >
-            <span class="nav-toggle-inner">
+            <label
-                <span class="nav-toggle-line"></span>
+                id="nav-toggle-label"
-                <span class="nav-toggle-line"></span>
+                for="nav-toggle"
-                <span class="nav-toggle-line"></span>
+                class="nav-toggle"
-            </span>
+                role="presentation"
-        </label>
+            >
+                <span class="nav-toggle-inner">
+                    <span class="nav-toggle-line"></span>
+                    <span class="nav-toggle-line"></span>
+                    <span class="nav-toggle-line"></span>
+                </span>
+            </label>
+        </button>
     </div>
     <div id="nav-menu" class="nav-menu container">
         <span class="nav-header-logo desktop">
@@ -149,6 +156,14 @@
         }
     }
 
+    .btn-clear {
+        background: transparent;
+        border: none;
+        padding: 0;
+        margin: 0;
+        cursor: pointer;
+    }
+
     .nav-toggle {
         display: none;
         align-items: center;

static/assets/js/mobile-menu-helper.js

@@ -1,8 +1,12 @@
 const MOBILE_BREAKPOINT_PX = 768;
 
+/** All focusable elements inside the menu (links). */
+const getMenuFocusables = (menu) =>
+    menu.querySelectorAll('a[href]');
+
 const mobileMenuHelper = () => {
     const mobileMenu = document.getElementById('nav-menu');
-    const mobileMenuToggleLabel = document.getElementById('nav-toggle-label');
+    const mobileMenuToggleButton = document.getElementById('nav-toggle-button');
     const mobileMenuToggle = document.querySelector('.nav-toggle-input');
     const menuItems = mobileMenu.querySelectorAll('.nav-item');
 
@@ -10,35 +14,75 @@ const mobileMenuHelper = () => {
 
     const syncMenuAriaHidden = () => {
         if (isMobile()) {
-            mobileMenu.setAttribute('aria-hidden', mobileMenuToggle.checked ? 'false' : 'true');
+            const hidden = !mobileMenuToggle.checked;
+            mobileMenu.setAttribute(
+                'aria-hidden',
+                hidden ? 'true' : 'false',
+            );
+            // inert removes the subtree from the a11y tree and makes descendants non-focusable
+            if (hidden) {
+                mobileMenu.setAttribute('inert', '');
+            } else {
+                mobileMenu.removeAttribute('inert');
+            }
+            setMenuFocusablesTabIndex();
         } else {
             mobileMenu.removeAttribute('aria-hidden');
+            mobileMenu.removeAttribute('inert');
+            getMenuFocusables(mobileMenu).forEach((el) => {
+                el.removeAttribute('tabindex');
+            });
         }
     };
 
-    const syncLabelAriaExpanded = () => {
+    const setMenuFocusablesTabIndex = () => {
-        mobileMenuToggleLabel.setAttribute('aria-expanded', mobileMenuToggle.checked ? 'true' : 'false');
+        const focusables = getMenuFocusables(mobileMenu);
+        const hidden = !mobileMenuToggle.checked;
+        focusables.forEach((el) => {
+            if (hidden) {
+                el.setAttribute('tabindex', '-1');
+            } else {
+                el.removeAttribute('tabindex');
+            }
+        });
+    };
+
+    const syncButtonAriaExpanded = () => {
+        mobileMenuToggleButton.setAttribute(
+            'aria-expanded',
+            mobileMenuToggle.checked ? 'true' : 'false',
+        );
     };
 
     menuItems.forEach((item) => {
         item.addEventListener('click', () => {
             mobileMenuToggle.checked = false;
-            syncLabelAriaExpanded();
+            syncButtonAriaExpanded();
             syncMenuAriaHidden();
         });
     });
 
     mobileMenuToggle.addEventListener('change', () => {
-        syncLabelAriaExpanded();
+        syncButtonAriaExpanded();
         syncMenuAriaHidden();
     });
 
+    mobileMenuToggleButton.addEventListener('keydown', (e) => {
+        if (e.key === 'Enter' || e.key === ' ') {
+            e.preventDefault();
+            mobileMenuToggle.checked = !mobileMenuToggle.checked;
+            // Programmatic .checked change does not fire 'change'; sync state so menu is focusable when open
+            syncButtonAriaExpanded();
+            syncMenuAriaHidden();
+        }
+    });
+
     window.addEventListener('resize', () => {
         syncMenuAriaHidden();
     });
 
     syncMenuAriaHidden();
-    syncLabelAriaExpanded();
+    syncButtonAriaExpanded();
 };
 
 document.addEventListener('DOMContentLoaded', mobileMenuHelper);

vite.config.ts

@@ -5,6 +5,12 @@ export default defineConfig({
     plugins: [sveltekit()],
     server: {
         host: true, // listen on 0.0.0.0 so reachable from host (e.g. dev container, port forward)
+        port: 5173,
+        strictPort: false,
+        // Disable HMR WebSocket when using port forwarding (e.g. dev container); the tunnel
+        // often doesn't proxy WebSockets, causing repeated connection failures in the console.
+        // With csr: false we don't use client-side HMR anyway—refresh the page to see changes.
+        hmr: false,
     },
     preview: {
         host: true,