Как я могу преобразовать этот грязный jQuery, чтобы быть менее раздутым и отзывчивым

StackOverflow https://stackoverflow.com/questions/19841802

  •  29-07-2022
  •  | 
  •  

Вопрос

Я использую jQuery Cycle2 и плагин Carousel для отображения некоторых событий на моем сайте. Все отлично работает, но я хочу, чтобы видимый вариант изменился с 5 на 3 на таблетках (между 768px и 1030px), а затем до 1 на телефонах (менее 768px). Все остальные варианты могут остаться прежними. Этот код взломан вместе и грязный, поэтому я ищу лучший способ сделать это. Кроме того, в настоящее время это работает только на обновлении. Это нормально, но было бы неплохо, если бы он перезагрузил и работал в режиме реального времени, когда вы изменили размер. Вот мой текущий код:

// Events
var ww = document.body.clientWidth;
$(document).ready(function() {
    adjustEvents();
})
$(window).bind('resize orientationchange', function() {
    ww = document.body.clientWidth;
    adjustEvents();
});

var adjustEvents = function() {
    if (ww > 1030) {
        $('.cycle').cycle({
            fx:'carousel',
            swipe:true,
            timeout:5000,
            slides:'> article',
            carouselVisible:5,
            carouselFluid:true,
            autoHeight:'calc',
            prev:'#prev',
            next:'#next'
        });
    } 
    else if (ww >= 768) {
        $('.cycle').cycle({
            fx:'carousel',
            swipe:true,
            timeout:5000,
            slides:'> article',
            carouselVisible:3,
            carouselFluid:true,
            autoHeight:'calc',
            prev:'#prev',
            next:'#next'
        });
    }
    else if (ww < 768) {
        $('.cycle').cycle({
            fx:'carousel',
            swipe:true,
            timeout:5000,
            slides:'> article',
            carouselVisible:1,
            carouselFluid:true,
            autoHeight:'calc',
            prev:'#prev',
            next:'#next'
        });
    }
}
Это было полезно?

Решение

$(document).ready(adjustEvents);
$(window).on('resize orientationchange', adjustEvents);

function adjustEvents() {
    var ww  = document.body.clientWidth,
        vis = ww > 1030 ? 5 : (ww >= 768 ? 3 : 1);
    $('.cycle').cycle({
        fx              : 'carousel',
        swipe           : true,
        timeout         : 5000,
        slides          : '> article',
        carouselVisible : vis,
        carouselFluid   : true,
        autoHeight      : 'calc',
        prev            : '#prev',
        next            : '#next'
    });
}

Другие советы

Вот «отделанная» версия кода ...

// Events
var ww = document.body.clientWidth;
$(document).ready(function() {
    adjustEvents();
})
$(window).bind('resize orientationchange', function() {
    ww = document.body.clientWidth;
    adjustEvents();
});

var adjustEvents = function() {
    var options = {
        fx:'carousel',
        swipe:true,
        timeout:5000,
        slides:'> article',
        carouselFluid:true,
        autoHeight:'calc',
        prev:'#prev',
        next:'#next'
    }

    if (ww > 1030) {
        options.carouselVisible = 5;
    } 
    else if (ww >= 768) {
        options.carouselVisible = 3;
    }
    else if (ww < 768) {
        options.carouselVisible = 1;
    }
    $('.cycle').cycle(options);
}

Все, что я действительно сделал, это создал переменную для хранения параметров цикла, и изменить только 1 свойство, которое имело отношение к каждой ширине.

Лицензировано под: CC-BY-SA с атрибуция
Не связан с StackOverflow
scroll top