следующем коде?
const modalWindow = document.getElementById('fadeBlock4');
const modalButton = document.getElementById('button-4');
const modalWindowButton = document.getElementById('ModalWindowButton4');
modalButton.onclick = function () {
modalWindow.classList.toggle('_active');
if (modalWindow.classList.contains("_active")) {
document.body.style.overflow = '';
} else {
document.body.style.overflow = 'hidden';
}
}
modalWindowButton.onclick = function () {
modalWindow.classList.toggle('_active');
if (modalWindow.classList.contains("_active")) {
document.body.style.overflow = '';
} else {
document.body.style.overflow = 'hidden';
}
}
(тут получается одна и таже функция для разных кнопок)
Вынеси повторяющийся код в 1 функцию и прикрепи её к событию
опачки, спасибо большое
у меня почему-то перестало работать((( function addingClassActive() { modalWindow.classList.toggle('_active'); if (modalWindow.classList.contains("_active")) { document.body.style.overflow = ''; } else { document.body.style.overflow = 'hidden'; } } modalWindowButton.onclick = addingClassActive() modalButton.onclick = addingClassActive()
addingClassActive без скобок
а, благодарю))
И лучше не пользоваться .onclick, так только 1 событие повесить 1 типа можно на элемент
Обсуждают сегодня