DaftAcademy / daftacademy-frontend4beginners-autumn2019

12 stars 18 forks source link

Problem z dodaniem event listenera #18

Open BoguszDmowski opened 4 years ago

BoguszDmowski commented 4 years ago

Mam problem- nie mogę ustawić event listenera dla mojego div'a o klasie page_hamburger zawierającego ikonę menu. Mój JS wygląda tak:

const page_hamburger = document.querySelector('.page-hamburger'); const navigationitems = document.querySelector('.page-navigation');

page-hamburger.addEventListener('click', () => {

navigationitems.classList.toggle('.page-navigationdwa');

});

Zadanie powinno być proste, a Ja męczę się z nim od trzech godzin. Jestem zielony z JS i na pewno nie widzę po prostu czegoś bardzo prostego :(

ZychGrzegorz commented 4 years ago

tak na szybko to w deklaracji stalej masz '_' , A dalej w kodzie masz '-' w nazwie stalej

ZychGrzegorz commented 4 years ago

Dodatkowo w nawiasie przy toggle usuń kropkę, w tym wypadku jeśli masz classlist to wiadomo, że operujesz na klasach i kropka jest błędem. Kropkę dajesz np. jeśli używasz queryselector, wtedy rozważane są zarówno klasy, id, jak i inne selektory I kropkę dajesz żeby zaznaczyć że szukasz po klasie.

karolryska commented 4 years ago

Trochę poza tematem - w JS dobrą praktyką jest używanie camelCase (notacja wielbłądzia) przy nazywaniu zmiennych. Przykładowo, zmienna zamiast "page-hamburger" nazywać się będzie "pageHamburger". Taki sposób zapisu możesz zauważyć też np. przy metodach - np. addEventListener czy classList.

BoguszDmowski commented 4 years ago

Bardzo dziękuję za pomoc :) działa