Поскольку вопрос был помечен jQuery, я предлагаю решение, использующее это.
Вы можете просмотреть его в jsFiddle .
Я изменил ваш код, Вот краткое описание того, что я изменил:
- Сначала я удалил встроенные атрибуты
onClick
и вместо этого использовал jQuery s $(element).on('click', function)
- Затем я добавил классы, такие как
.open-modal
и .close-modal
, где это уместно. При таком подходе вам не нужно передавать идентификатор элемента, вы можете просто получить контент, ближайший к нажатой кнопке. - Я добавил класс
.modal-info
в информационный контейнер, чтобы упростить таргетинг . - Наконец, я обработал все открытия / закрытия с помощью jQuery.
HTML:
<div id="myModal" class="myModal">
<button class="close-modal">×</button>
<div id="modalContent">
</div>
</div>
<div class="description">
<h2>Exercitation</h2>
<p class="open-modal">Read more...</p>
<div id="1" class="modal-info">
<h2>#1 Exercitation ullamco laboris nisi</h2>
<hr>
<p><strong>adipisicing elit</strong></p>
<p>Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod
tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam,
quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo
consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse
cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non
proident, sunt in culpa qui officia deserunt mollit anim id est laborum.</p>
</div>
</div>
<div class="description">
<h2>Exercitation</h2>
<p class="open-modal">Read more...</p>
<div id="2" class="modal-info">
<h2>#2 Exercitation ullamco laboris nisi</h2>
<hr>
<p><strong>adipisicing elit</strong></p>
<p>Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod
tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam,
quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo
consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse
cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non
proident, sunt in culpa qui officia deserunt mollit anim id est laborum.</p>
</div>
</div>
jQuery:
$('.open-modal').on('click', open_modal);
$('.close-modal').on('click', close_modal);
function open_modal() {
// Modal container
var $modal = $('#myModal');
// Get content container
var $modal_content = $('#modalContent');
// Empty existing content first
$modal_content.empty();
// Get new content
// $(this) is the .open-modal element that was clicked
// I'm using jQuery siblings() to get the nearest .modal-info element
var new_modal_content = $(this).siblings('.modal-info').html();
// Set new content
$modal_content.html(new_modal_content);
// Display modal
$modal.show();
}
function close_modal(){
var $modal = $('#myModal');
if($modal.is(':visible')){
$modal.hide();
}
}
CSS:
#myModal {
display: none;
}
.modal-info {
display: none;
}
Хотя это не идеально и может быть улучшено. Тем не менее, я думаю, что легче поддерживать, чем встроенные атрибуты onClick, а что нет.