Почему мой медиа-запрос на боковой панели не работает в Javascript?

Question

У меня есть боковая панель, которая выдвигается на 250 пикселей с использованием JavaScript на рабочем столе. Когда дело доходит до мобильного представления, я хочу, чтобы боковая панель занимала 100% ширины. Я пытаюсь использовать Media Queries в Javascript, но неважно, какие изменения я делаю. Кажется, мои стили меняются поверх моей боковой панели на рабочем столе.

HTML

 <nav class="navbar">
  <div id="toggle-btn" class="sidemenu-btn">
    <i class="fas fa-bars"></i>
  </div>
  <div id="toggle-nav" class="navbar-items animated fadeInLeft delay-1s">
    <a href="#home">Home</a>
    <a href="#about-me">About</a>
    <a href="#my-skills">Skills</a>
    <a href="#my-portfolio">Portfolio</a>
    <a href="#contact-me">Contact</a>
  </div>
</nav>

CSS

 .navbar {
      height: 100%;
      width: 0;
      position: fixed;
      background: #141313;
    }

    .navbar .sidemenu-btn {
     font-size: 2.5rem;
     padding: 3rem 0;
     text-align: center;
     margin-left: 1rem;
     cursor: pointer;
     color: #141313;
    }

    .navbar .navbar-items {
    display: flex;
    flex-direction: column;
    text-align: center;
    display: none;
    margin-left: 1rem;
    }

   .navbar .navbar-items a {
    text-decoration: none;
    color: white;
    padding: 1.2rem 0;
    font-weight: 400;
    font-size: 1.2rem;
    text-transform: uppercase;
    }

   .navbar .navbar-items a:hover {
    text-decoration: line-through;
   }

@media screen and (max-width: 768px) {

.navbar {
  position: relative;
}


}

JS

const toggleBtn = document.querySelector("#toggle-btn");
const toggleNav = document.querySelector(".navbar");
const togglenavItems = document.querySelector('.navbar-items');

toggleBtn.addEventListener("click", sideMenu);
toggleBtn.addEventListener("click", mediaQuery);


    function sideMenu() {
  if (toggleNav.style.width === "250px") {
    toggleNav.style.width = "0px";
  } else {
    toggleNav.style.width = "250px";
  }

}



function mediaQuery() {
  const x = window.matchMedia('(max-width: 768px)');
  const y = document.querySelector('.navbar');

  if (x.matches && y.style.width === "100%") {
    y.style.width = "0px";

  } else {
    y.style.width = "100%";

  }



}

Answer 1

Это проще сделать с помощью CSS, чтобы управлять тем, как он выглядит на мобильном / настольном компьютере, а JS просто обрабатывает логику.Следующее исправит несоответствие стилей и облегчит навигацию в JS.

CSS

 .navbar{
   width: 250px;
   height: 100%;
   position: fixed;
   background: #141313;
 }     
 .navHidden{
  width: 0px !important;
 }
 /* smaller screen size */
 @media screen and (max-width:768px) {
   .navbar{
     width: 100%;
   }
 }

HTML

<nav class="navbar navHidden">

JS

function sideMenu() {
  toggleNav.classList.toggle("navHidden")
}

Sidenote

Во включенном коде щелчком вашей кнопки-переключателя вы будете устанавливать ширину навигационной панели равной 250px , но затем она будет передана следующей функции,который затем поменяет его на 100% в КАЖДЫЙ СЛУЧАЙ независимо от размера экрана, поэтому написанная вами функция переключения никогда не будет работать.

Answer 2

Медиа-запросы могут сделать работу. Вам не нужны js для изменения ширины navbar-items. Посмотреть это

const toggleBtn = document.querySelector("#toggle-btn");
const toggleNav = document.querySelector(".navbar");
const togglenavItems = document.querySelector('.navbar-items');

toggleBtn.addEventListener("click", sideMenu);

function sideMenu() {
  toggleNav.classList.toggle('open');
}

* {
  box-sizing: border-box;
}

body {
  margin: 0;
}

.navbar {
    height: 100%;
    width: 100%;
    position: fixed;

  }

  .sidemenu-btn {
     font-size: 2.5rem;
     padding: 0;
     text-align: center;
     cursor: pointer;
     color: green;
     position: absolute;
     right: 20px;
     top: 20px;
  }

  .navbar-items {
    display: flex;
    flex-direction: column;
    text-align: center;
    width: 250px;
    height: 100%;
    padding-left: 1rem;
    background: #141313;
    transition: all .5s ease;
  }

 .navbar .navbar-items a {
    text-decoration: none;
    color: white;
    padding: 1.2rem 0;
    font-weight: 400;
    font-size: 1.2rem;
    text-transform: uppercase;
  }

 .navbar .navbar-items a:hover {
  text-decoration: line-through;
 }


@media (max-width: 768px) {
  .navbar-items {
    width: 100%;
  }
}

.navbar.open .navbar-items {
  width: 0;
}

<nav class="navbar">
  <div id="toggle-btn" class="sidemenu-btn">
    <i class="fas fa-bars"></i>
    Toggle
  </div>
  <div id="toggle-nav" class="navbar-items animated fadeInLeft delay-1s">
    <a href="#home">Home</a>
    <a href="#about-me">About</a>
    <a href="#my-skills">Skills</a>
    <a href="#my-portfolio">Portfolio</a>
    <a href="#contact-me">Contact</a>
  </div>
</nav>

Просто добавив медиа-запрос, вы можете изменить ширину элемента, нет необходимости проверять в js

@media (max-width: 768px) {
 .navbar-items {
   width: 100%;
 }
}

Добавление класса .open, который будет добавлен к .navbar и вызовет переход

.navbar.open .navbar-items {
  width: 0;
}

Ваш JS теперь упрощен

const toggleBtn = document.querySelector("#toggle-btn");
const toggleNav = document.querySelector(".navbar");

toggleBtn.addEventListener("click", sideMenu);

function sideMenu() {
  toggleNav.classList.toggle('open');
}