Почему модульное окно не закрывается, по клику по крестику?

Question

[dsfafdagsadf]

Вот код
https://jsfiddle.net/dLoqjk7h/

Comments

[0xD34F]

6.5

[dsfafdagsadf]

0xD34F, чего?

[0xD34F]

dsfafdagsadf, сколько тебя раз уже банили? 30? 50? Всё никак не дойдёт, что правила нарушать не надо?

[dsfafdagsadf]

0xD34F, а я и не нарушаю, что тебе нужно я не пойму.?
что тебе нужно, что тебе нужно от меня?????????????????????????????????????????????
отстань

[WapSter]

0xD34F, думаешь у человека с таким ником это единственный аккаунт? По моему они вообще сюда заходят с вопросами из заданий с фриланса.

[0xD34F]

dsfafdagsadf, нарушаешь. Использовать более одного аккаунта запрещено.

[dsfafdagsadf]

0xD34F, у меня один, у меня один,
ты что за мной следишь? ты нарушаешь мои конституционные права,

Answer 1

[Александр]

event.target == modal меняем на event.target.closest('.modal') в данном случае.

Или убираем window.onclick. document.getElementsByClassName возвращает массив элементов, поэтому либо span[0].onclick либо var span = document.querySelector('.close');.

Answer 2

[Степан Крапивин]

потому что

var span = document.getElementsByClassName("close");

возвращает HTMLCollection, т.е. не один элемент а список элементов.

А дальше идёт навешивание обработчика клика как для единственного элемента:

span.onclick = function() {
  modal.style.display = "none";
}

нужно сначала взять элемент из HTMLCollection и на него уже навешивать обработчик

span[0].onclick = function() {
  modal.style.display = "none";
}

или в цикле обходить все элементы HTMLCollection и на каждый навешивать обработчик