-1

Подскажите пожалуйста. имеются три переменные, для которых есть повторяющиеся действия. Каким образом упростить написание или убрать дублирование кода?

$('.main_btna').on('click', function() {
    $('.overlay').fadeIn();
    $('.modal').animate({
        opacity: 'show',
        top: '0'
    },
        1000);
});

$('.main_btn').on('click', function() {
    $('.overlay').fadeIn();
    $('.modal').animate({
        opacity: 'show',
        top: '0'
    },
        1000);
});

$('.schedule').on('click', function() {
    $('.overlay').fadeIn();
    $('.modal').animate({
        opacity: 'show',
        top: '0'
    },
        1000);
});
1
  • 2
    вынести функцию. В on указывать имя функции
    – ArchDemon
    2 мая 2020 в 15:09

1 ответ 1

2

Сделай общий селектор, вот так:

$('.main_btna, .main_btn, .schedule').click(()=>{
    $('.overlay').fadeIn();
    $('.modal').animate({
        opacity: 'show',
        top: '0'
    }, 1000);
});

+Если у тебя всего один .overlay [и\или] .modal, лучше дописать к ним :first (без пробела), чтоб jq не искал по древу эл-ты

1
  • А ещё лучше сделать класс .js-modal например, и повесить их на все элементы, которые будут выполнять код, ну а в коде указать $('.js-modal')
    – De.Minov
    5 авг 2022 в 7:11

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.