При выборе вещей вы можете использовать
document.querySelector( query:String ) // returns an HTMLElement or null
document.querySelectorAll( query:String ) // returns a NodeList you can iterate over
Для переключения классов вы можете использовать classList
и его методы для управления классами на HTMLElements:
HTMLElement.classList.toggle( className:String, forced:Boolean|Null )
И для добавленияСлушатели событий, просто используйте addEventListener
вместо on
.
Это превращает ваш код в это:
const menuIconEl = document.querySelector('.menu-icon');
const sidenavEl = document.querySelector('.sidenav');
const sidenavCloseEl = document.querySelector('.sidenav-close-icon');
const onclick = e => sidenavEl.classList.toggle( 'active' );
menuIconEl.addEventListener( 'click', onclick );
sidenavCloseEl.addEventListener( 'click', onclick );
И это делает!