IsayR

Сделайте одинаковый position у header и submenu1.
Было похожая проблема, решил, поставив одинаковое значение свойства position.

Я вообще ещетолько учусь но яб попробывал все твои кнопки найти через queryselectorAll и перебрать этот массив через forEatch() навешивая уже события клика.?

про id замечание верное.
кароче:
в css
добавь класс .close{display:none;}

в js

1 строку меняем на const btn = document.querySelectorAll('.btn-modal')

затем:
btn.forEach(btnForm => {
btnForm.addEventListener('click', (e) => {
modal.classList.remove('close');
modal.classList.add('active');
});
});

const closeModal = () => {
modal.classList.remove('open');
modal.classList.add('close');
}