$(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'
});
}
¿Cómo puedo convertir esta desordenada jQuery para que esté menos hinchada y receptiva?
-
29-07-2022 - |
Pregunta
Estoy usando JQuery Cycle2 y el complemento Carousel para mostrar algunos eventos en mi sitio. Todo funciona muy bien, pero quiero que la opción visible cambie de 5 a 3 en tabletas (entre 768px y 1030px) y luego hasta 1 en teléfonos (menos de 768px). Todas las otras opciones pueden seguir siendo las mismas. Este código está pirateado y desordenado, así que estoy buscando una mejor manera de hacerlo. Además, actualmente solo funciona en actualización. Eso está bien, pero sería bueno si se volviera a cargar y funcionara en tiempo real cuando cambia de tamaño. Aquí está mi código actual:
// 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'
});
}
}
Solución
Otros consejos
Aquí hay una versión "recortada" del código ...
// 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);
}
Todo lo que realmente hice fue crear una variable para almacenar las opciones de ciclo y solo cambiar la propiedad 1 que era relevante para cada ancho.