подменю при клике на элемент меню, насколько правильно написано

[13Foch]

куда бы его засунуть тут, не подскажите? )))

да тут никуда не всунешь потому что this это .menu а не .menu__item

[13Foch]

На светофоре стоит Ламборджини... Рядом останавливается Логан.
Водила Логана делает знаки водиле Ламборджини, де есть вопрос к нему... Водила Ламборджини опускает стекло:
- Что хотел?
- Хотел узнать как тачка?
- Нормальная.
- Не ломается?
- Нет. А в чем проблема?
- Да вот смотрю не особо ее народ покупает...

я понимаю мой код для вас глупым кажется )), но спасибо мне полезна любая информация негативная тоже пока катаюсь на логане )

[ksa]

13Foch, просто всему свое место... Так и this используется там, где он нужен.

[13Foch]

13Foch, просто всему свое место... Так и this используется там, где он нужен.

Понятно

[13Foch]

куда бы его засунуть тут, не подскажите? )))

благодаря вашему вопросу посмотрел как работает target и на этом спасибо раньше не знал

[рони]

13Foch,

[13Foch]

let a = document.querySelector(".menu");
		if (a) {
			a.addEventListener("click", function (event) {
				let target = event.target;
				if (target = target.closest(".menu__name")) {
					event.preventDefault();
					let parent = target.parentNode;
					let children = parent.parentNode.children;
					[...children].forEach(el => el.classList[el === parent ? "toggle" : "remove"]("menu__item--active"))
				}
			});
		}

а так нормально писать ? а то вдруг на следующей странице не будет меню этого получиться ошибка

[рони]

а так нормально писать ?

да

[рони]

а то вдруг на следующей странице не будет меню этого получиться ошибка

можно клик не на меню ставить в данном случае а на document, и тогда никаких проверок.
но ваш код более правильный, это приём делегирование, лучше делать на ближайший общий предок и не грузить лишние обработчики.

[13Foch]

понял