Как объединить и сократить этот код?

Question

[Eugene Chefranov]

Имеется код который выполняет одну и ту же функцию - закрытие поп-апа, по клику на крестик или по нажатию на Esc. Как можно объединить этот в один и тем самым сократить код?

$('.modal-close').click(function (e) {
         e.preventDefault();
         $('.modal-window').removeClass('active');
         $('.modal-overlay').removeClass('active');
         $('body').css('overflow-y', 'auto');
     });

     $(this).keydown(function (eventObject) {
         if (eventObject.which == 27) {
             $('.modal-window').removeClass('active');
             $('.modal-overlay').removeClass('active');
             $('body').css('overflow-y', 'auto');
         }
     });

Comments

[msdos-x86]

Можно повторяющиеся три строки в функцию вынести

Answer 1

[Владимир Проскурин]

Как минимум

function closeModal() { 
    $('.modal-window, .modal-overlay').removeClass('active');
    $('body').css('overflow-y', 'auto');
}
$('.modal-close').click(function (e) {
     e.preventDefault();
     closeModal();
});

$(this).keydown(function (eventObject) {
     if (eventObject.which == 27) {
         closeModal();
     }
});