Dmitry0900

e.target возвращает нажатый элемент, а не модальное окно.
window.onclick = function(event) {
if (event.target.closest("secondary") === modalparent[...]) {
modalparent[...].classList.remove( 'active' );
}
};