Removing hrefs like this is breaks a11y entirely

src/templates/src/include/header.html

@@ -139,9 +139,9 @@
             const loginLink = document.getElementById("login-link");
 
             if (loginPopup !== null) {
-                loginLink.removeAttribute("href");
 
-                loginLink.onclick = () => {
+                loginLink.onclick = (e) => { 
+                    e.preventDefault();
                     loginPopup.classList.toggle("open");
                 }
             }