HTML, CSS, JS

Community feedback

• 

  Abhik•4,840

  @abhik-bPosted almost 4 years ago

  Good job Shade , your solution looks nice. Keep it up 💯

  You can do something like this to reduce creating & calling a extra function (just a opinion)

  hamburger.addEventListener("click", () => {
      mobileMenu.style.display = "block";
      if (hamburger.dataset.status === "open") {
          hamburger.src = "./images/icon-close.svg";
          hamburger.dataset.status = "close";
          mobileMenu.classList.add("fadeInMobileMenuAnimation");
          mobileMenu.classList.remove("fadeOutMobileMenuAnimation");
      } else {
          hamburger.src = "./images/icon-hamburger.svg";
          hamburger.dataset.status = "open";
          mobileMenu.classList.add("fadeOutMobileMenuAnimation");
          mobileMenu.classList.remove("fadeInMobileMenuAnimation");
      }
  
  });
  
  

  0