Removed all references to redundant 'padding' class on primary naviga…

…tion js and html
chrisnajman · Jan 11, 2024 · 6318f18 · 6318f18
1 parent 6bc919c
commit 6318f18
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/index.html b/index.html
@@ -36,7 +36,7 @@
             <h1>Accessible Mobile Menu</h1>
             <p>To interact with the mobile menu, narrow the window width to 768px or less.</p>
         </header>
-        <div id="primary-navigation" class="primary-navigation padding">
+        <div id="primary-navigation" class="primary-navigation">
             <div class="hamburger-button-wrapper">
                 <h2>Menu</h2>
                 <button class="hamburger-button" id="hamburger-button" aria-controls="site-navigation"

diff --git a/js/primary-nav-links.js b/js/primary-nav-links.js
@@ -1,9 +1,4 @@
-import {
-  hamburgerButton,
-  btnTxt,
-  primaryNavigation,
-  siteNavigation,
-} from "./globals.js"
+import { hamburgerButton, btnTxt, siteNavigation } from "./globals.js"
 
 export default function primaryNavLinks() {
   const navLinks = document.querySelectorAll(".nav-link")
@@ -14,7 +9,6 @@ export default function primaryNavLinks() {
       btnTxt.textContent = "Expand menu"
       siteNavigation.classList.add("menu-hidden")
       siteNavigation.classList.remove("menu-visible")
-      primaryNavigation.classList.remove("padding")
     })
   })
 }