javascript - 将 'active' 类添加到幻灯片导航

Question

我创建了一个幻灯片，可以使用 jQuery 按一定间隔淡入和淡出图像。我在底部创建了一个导航区域，当您单击一个正方形时，它会显示该幻灯片。有两件事我需要帮助。

当您单击导航方 block 时，计时会关闭，直到它转到下一个自然幻灯片。

我需要将当前幻灯片的方 block 设置为不同的颜色。我一直在尝试将一个类添加到具有匹配 ID 的正方形中，但它没有添加类。

    $("#slideshow1 > div:gt(0)").hide();

var newvar = setInterval(function() { 
    var currentslide = $('.current').attr('id');
    $('#slideshow1 > div:first')
        .fadeOut(1000)
        .next()
        .addClass('current')
        .fadeIn(1000)
        .end()
        .removeClass('current')
    .appendTo('#slideshow1');
    $('.slider-nav').removeClass('active');
    $('.slider-nav[data-number=' + currentslide + ']').addClass('active');
},  5000);

$(".slider-nav").click(function(e){
    e.preventDefault();
    var currentID = $(this).attr('href');
    $('#slideshow1 > div:first').fadeOut(1000);
    $('div'+currentID).fadeIn(1000).nextAll("div.slide").addBack().prependTo("#slideshow1");
    $('.slider-nav').removeClass('active');
    $('.slider-nav[data-number=' + currentID + ']').addClass('active');
    clearInterval(newvar);
    newvar = setInterval(function() { 
        $('#slideshow1 > div:first')
            .fadeOut(1000)
            .next()
            .addClass('current')
            .fadeIn(1000)
            .end()
            .removeClass('current')
        .appendTo('#slideshow1');
        $('.slider-nav').removeClass('active');
        $('.slider-nav[data-number=' + currentID + ']').addClass('active');
    },  5000);
});

此幻灯片所在的网站:http://saratoga.mspaceap.com/

Answer 1

您的问题在这里:

$('.slider-nav[id=' + currentID + ']').addClass('active');

因为 currentID 是一个类似于 #1 的字符串，因为您之前将其设置为三行:

var currentID = $(this).attr('href');

要修复，请将代码更改为:

$('.slider-nav[id=' + currentID.substr(1) + ']').addClass('active')
    .siblings().removeClass('active'); // don't forget to do this

---

也就是说:您的导航元素和幻灯片不应具有相同的 ID。 ID 应该始终是唯一的。我可以看到您正在使用 .attr('id') 解决此问题，但这仍然是不好的做法。请使用常见的 data- 属性或像 nav1 这样的 id。