Tried to convert simple jquery code to vanilla js

[atheneor]

Hi guys, 

I tried to convert the following jquery code to vanilla js. It seems like I've made mistake. 

$(".menu-link").click(function() {
  $("#menu").toggleClass("active");
  $(".container").toggleClass("active");
});

document.querySelectorAll(".menu-link").addEventListener("click", () => {
  var menu = document.querySelector("#menu");
  menu.classList.toggle("active");
  var container = document.querySelector(".container");
  container.classList.toggle("active");
});

Thanks a lot!

[gebeer]

Hello, and welcome to the forum. This is not really a ProcessWire specific question. You might get advice or even find a solution on stackoverflow.com. And http://youmightnotneedjquery.com/ also might have some useful info.

[BitPoet]

9 hours ago, atheneor said:

It seems like I've made mistake. 

Just one, I think. document.querySelectorAll returns a NodeList. Unlike with jquery collections, you can't call addEventListener on the NodeList but rather need to iterate over its elements and attach a listener to each one.

Array.forEach.call(nodeList, function(el) { el.addEventListener(/* your class toggle code here */); })