From ebd98b1f3470de52596de5f4fabe3610dfc26863 Mon Sep 17 00:00:00 2001
From: Cory Dransfeldt <hi@coryd.dev>
Date: Thu, 20 Jun 2024 09:35:31 -0700
Subject: [PATCH] chore: tidy up markup

---
 package.json                           | 2 +-
 src/_includes/partials/nav/menu.liquid | 6 ++----
 src/assets/scripts/menu.js             | 2 +-
 src/assets/styles/components/menu.css  | 4 ----
 4 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/package.json b/package.json
index 33c4da46..05518822 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "coryd.dev",
-  "version": "19.5.10",
+  "version": "19.5.11",
   "description": "The source for my personal site. Built using 11ty.",
   "type": "module",
   "scripts": {
diff --git a/src/_includes/partials/nav/menu.liquid b/src/_includes/partials/nav/menu.liquid
index fb067d0f..5dd21cef 100644
--- a/src/_includes/partials/nav/menu.liquid
+++ b/src/_includes/partials/nav/menu.liquid
@@ -4,16 +4,14 @@
 <script>{{ js }}</script>
 <div class="flex-centered">
   <input id="menu-toggle" type="checkbox" aria-hidden="true" />
-  <label class="menu-button-container" for="menu-toggle" aria-controls="primary-navigation" aria-expanded="false" tabindex="0">
+  <label class="menu-button-container" for="menu-toggle" aria-expanded="false" tabindex="0">
     <span id="menu-label-text" class="screen-readers-only">Toggle mobile menu</span>
     <div class="menu-closed" aria-hidden="true">{% tablericon "menu" "Menu closed" %}</div>
     <div class="menu-open" aria-hidden="true">{% tablericon "x" "Menu open" %}</div>
   </label>
   <ul class="menu-primary" aria-label="Primary site navigation" id="primary-navigation" role="menu">
     {% for link in nav.menu %}
-    <li role="menu-item">
-      {% render "partials/nav/link.liquid", page:page, link:link.name, icon:link.icon %}
-    </li>
+      <li>{% render "partials/nav/link.liquid", page:page, link:link.name, icon:link.icon %}</li>
     {% endfor %}
   </ul>
   {% render "partials/nav/theme-toggle.liquid" %}
diff --git a/src/assets/scripts/menu.js b/src/assets/scripts/menu.js
index 4cc9ba03..6b4f151d 100644
--- a/src/assets/scripts/menu.js
+++ b/src/assets/scripts/menu.js
@@ -2,7 +2,7 @@ window.addEventListener('load', () => {
   const menuInput = document.getElementById('menu-toggle')
   const menuLabelText = document.getElementById('menu-label-text')
   const menuButtonContainer = document.querySelector('.menu-button-container')
-  const menuItems = document.querySelectorAll('.menu-primary li[role="menu-item"]')
+  const menuItems = document.querySelectorAll('.menu-primary li')
   const isMobile = () => window.innerWidth <= 768
 
   const udpateMenuState = () => {
diff --git a/src/assets/styles/components/menu.css b/src/assets/styles/components/menu.css
index 60e3d4ac..96d3df7d 100644
--- a/src/assets/styles/components/menu.css
+++ b/src/assets/styles/components/menu.css
@@ -73,10 +73,6 @@
   }
 }
 
-a[role="menu-item"]:focus {
-  outline: none;
-}
-
 @media (max-width: 768px) {
   .menu-primary {
     position: absolute;