@@Matthias Scharwies
Danke, ich hab's kopiert und versuch's mal in das JS-Tutorial einzubauen.
Da besteht noch einiges Verbesserungspotential. this
anstatt event.target
hatte ich ja schon erwähnt, aber auch damit wäre
event.target.parentNode.classList.toggle("open");
const open = event.target.parentNode.classList.contains("open");
noch unsinnig. Element.classList.toggle()
gibt ja schon genau das zurück, was man haben möchte. Die Abfrage mit Element.classList.contains()
ist Quatsch.
const open = this.parentNode.classList.toggle("open");
und gut ist.
LLAP 🖖
--
“When UX doesn’t consider all users, shouldn’t it be known as ‘Some User Experience’ or... SUX? #a11y” —Billy Gregory
“When UX doesn’t consider all users, shouldn’t it be known as ‘Some User Experience’ or... SUX? #a11y” —Billy Gregory