Работа onclick на много элементов?

Question

[Сергей Кирьянов]

На странице есть много элементов, которые должны закрывать всплывающее окно:

<div class="someclass1">
<button class="mfp-close"><img class="mfp-close-img" src="/url/" alt="Закрыть"></button>
</div>
<div class="someclass2">
<button class="mfp-close"><img class="mfp-close-img" src="/url/" alt="Закрыть"></button>
</div>
<div class="someclass3">
<button class="mfp-close"><img class="mfp-close-img" src="/url/" alt="Закрыть"></button>
</div>

Закрытие происходит этим кодом

$( ".mfp-close-img" ).click(function() {
    var magnificPopup = $.magnificPopup.instance;
    magnificPopup.close();
    console.log('close must work');
});

но работает только закрытие одного окна, как повесить его на все с классом mfp-close-img?

Comments

[Павел]

все должно работать, показывай весь код

Answer 1

[Сергей Кирьянов]

Проблема решилась изменением на

$(document).on('click', '.mfp-close-img', function () {

});

Comments

[Михаил Макаров]

Честно говоря странно, что этот код решил проблему, а предыдущий - не работал, как нужно. Из того, что ты написал, и предыдущий должен был правильно отрабатывать.

[FeST1VaL]

Михаил Макаров, Скорее всего окно создавалось и по этому там не было навешано событие и он не работал)

[Сергей Кирьянов]

Артем Будин, да, создавалось окно