From 3d428bc99a49c89210bf86bbd71a1b8ff8b24f9d Mon Sep 17 00:00:00 2001
From: Cory Dransfeldt <hi@coryd.dev>
Date: Thu, 27 Jun 2024 14:04:17 -0700
Subject: [PATCH] chore: cleanup

---
 src/_includes/partials/nav/menu.liquid        |  1 -
 .../partials/nav/theme-toggle.liquid          |  2 +-
 src/assets/scripts/menu.js                    | 20 +-----------
 src/assets/styles/base/index.css              | 31 -------------------
 4 files changed, 2 insertions(+), 52 deletions(-)

diff --git a/src/_includes/partials/nav/menu.liquid b/src/_includes/partials/nav/menu.liquid
index 1640855d..f4a7a563 100644
--- a/src/_includes/partials/nav/menu.liquid
+++ b/src/_includes/partials/nav/menu.liquid
@@ -5,7 +5,6 @@
 <div class="flex-centered">
   <input id="menu-toggle" type="checkbox" aria-hidden="true" />
   <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>
diff --git a/src/_includes/partials/nav/theme-toggle.liquid b/src/_includes/partials/nav/theme-toggle.liquid
index ff34a28b..a86b71d5 100644
--- a/src/_includes/partials/nav/theme-toggle.liquid
+++ b/src/_includes/partials/nav/theme-toggle.liquid
@@ -1,7 +1,7 @@
 <script type="module" src="/assets/scripts/components/theme-toggle.js" crossorigin="anonymous"></script>
 <span class="client-side">
   <theme-toggle>
-    <button class="theme-toggle" aria-label="Toggle site theme" tabindex="0">
+    <button class="theme-toggle" tabindex="0">
       <span class="light">
         {% tablericon "sun" "Toggle light theme" %}
       </span>
diff --git a/src/assets/scripts/menu.js b/src/assets/scripts/menu.js
index 6b4f151d..fc292560 100644
--- a/src/assets/scripts/menu.js
+++ b/src/assets/scripts/menu.js
@@ -1,27 +1,12 @@
 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')
-  const isMobile = () => window.innerWidth <= 768
-
-  const udpateMenuState = () => {
-    const isExpanded = menuInput.checked
-    menuButtonContainer.setAttribute('aria-expanded', isExpanded)
-
-    if(isExpanded) menuLabelText.textContent = 'Close mobile menu'
-    if (!isExpanded) menuLabelText.textContent = 'Open mobile menu'
-  }
-
-  udpateMenuState()
-
-  menuInput.addEventListener('change', udpateMenuState)
 
   menuButtonContainer.addEventListener('keydown', e => {
     if (e.key === 'Enter' || e.key === ' ') {
       e.preventDefault()
       menuInput.checked = !menuInput.checked
-      udpateMenuState()
     }
   })
 
@@ -35,9 +20,6 @@ window.addEventListener('load', () => {
   })
 
   document.addEventListener('keydown', e => {
-    if (e.key === 'Escape' && isMobile() && menuInput.checked) {
-      menuInput.checked = false
-      udpateMenuState()
-    }
+    if (e.key === 'Escape' && menuInput.checked) menuInput.checked = false
   })
 })
\ No newline at end of file
diff --git a/src/assets/styles/base/index.css b/src/assets/styles/base/index.css
index bc94707f..4ba3a901 100644
--- a/src/assets/styles/base/index.css
+++ b/src/assets/styles/base/index.css
@@ -561,37 +561,6 @@ li {
   display: none !important;
 }
 
-.hidden-visually {
-  border: 0;
-  clip: rect(0 0 0 0);
-  height: 1px;
-  margin: -1px;
-  overflow: hidden;
-  padding: 0;
-  position: absolute;
-  white-space: nowrap;
-  width: 1px;
-
-  &:focus {
-    clip: auto;
-    height: auto;
-    margin: 0;
-    overflow: visible;
-    position: static;
-    width: auto;
-  }
-}
-
-.screen-readers-only:not(:focus):not(:active) {
-  clip: rect(0 0 0 0);
-  clip-path: inset(50%);
-  height: 1px;
-  overflow: hidden;
-  position: absolute;
-  white-space: nowrap;
-  width: 1px;
-}
-
 .text-centered {
   text-align: center !important;
 }