Всячески стараюсь писать более менее грамотный JS. Пока получается не очень, но с Вашей помощью продуктивность моих потуг должна возрасти.
Начал недавно, не судите строго. JQuery стараюсь всячески избегать дабы освоить именно JS.
Вот мой код, который я пытаюсь улучшить:
function showModal() {
var btn = document.getElementById('callback-btn');
var modal = document.getElementById('modalCallback');
btn.onclick = function(){
modal.classList.add('open');
}
modal.onclick = function(){
if (modal.classList.contains('open')) {
modal.classList.remove('open');
}
}
}
showModal();
По клику на элемент с id callback-btn, элементу с id modalCallback добавляется класс open. В общем скрипт свою функцию выполнят, однако не выделяется изящностью и гибкостью.
Так как на сайте может быть более одной формы правильно было бы не строго задавать id элемента, а получать его динамически. Сохранять результат в переменную и обрабатывать по разному, в зависимости от значения.
Я думаю, что стоит разделить представление и логику. Однако усложнять до полноценно MVC не вижу смысла.
Как вы считаете?
