Во-первых, вы должны установить обе кнопки закрытия как имеющие один и тот же класс, поэтому class="close"
.
Также используйте одну функцию для закрытия модального диалога, такую как
function closeModal() {
modal.style.display = "none";
}
Затем замените следующие
var span = document.getElementsByClassName("close")[0];
// When the user clicks on <span> (x), close the modal
span.onclick = function() {
modal.style.display = "none";
}
на
var spans = document.getElementsByClassName("close");
for (let closeBtn of spans) {
closeBtn.onclick = closeModal;
}
Причина, по которой одна из кнопок закрытия перестает работать, заключается в том, что document.getElementsByClassName()
возвращает HTMLCollection , которая представляет собой коллекция найденных элементов. Также вызов getElementsByClassName("close")[0]
возвращает первый элемент в коллекции, и только одна кнопка закрытия получает событие onclick
. Чтобы исправить это, решение состоит в том, чтобы получить не только первый элемент, но и все элементы, затем выполнить итерацию по всем из них и добавить обработчик onclick
ко всем элементам, как показано.
Атрибут class
в отличие от id
атрибут не обязательно должен быть уникальным и поэтому оба могут иметь одинаковое значение close
.
Полный рабочий пример:
<!-- The Modal -->
<div id="myModal" class="modal">
<!-- Modal content -->
<div class="modal-content">
<span class="close">×</span>
<a href="#" class="close">close</a>
<p>Some text in the Modal..</p>
</div>
</div>
<script>
function closeModal() {
modal.style.display = "none";
}
// Get the modal
var modal = document.getElementById("myModal");
// Get the button that opens the modal
var btn = document.getElementById("myBtn");
// Get the <span> element that closes the modal
var spans = document.getElementsByClassName("close");
for (let closeBtn of spans) {
closeBtn.onclick = closeModal;
}
// When the user clicks the button, open the modal
btn.onclick = function() {
modal.style.display = "block";
}
// When the user clicks anywhere outside of the modal, close it
window.onclick = function(event) {
if (event.target == modal) {
modal.style.display = "none";
}
}
</script>