This forum is now read-only. Please use our new forums! Go to forums

banner
Close banner
0 points
Submitted by rose8779
over 9 years

"Remember to remove '.active-dot' from the current dot";Can't Find Problem in Code

I can’t find why the program keeps on telling me to “remember to remove ‘.active-dot’ from the current dot.” I have changed my code numerous times but I keep on getting the same message. If anyone could help me, that would be great. My code: var main = function(){ $(‘.dropdown-toggle’).click(function(){ $(‘.dropdown-menu’).toggle(); }); $(‘.arrow-next’).click(function(){ var currentSlide = $(‘.active-slide’); var nextSlide = currentSlide.next;

    var currentDot = $('.active-dot');    
    var nextDot = currentDot.next();
   
    if(nextSlide.length ==0){
        nextSlide=$('.slide').first();
        nextDot=$('.dot').first();}
        
   currentSlide.fadeOut(600).removeClass('active-slide');
    nextSlide.fadeIn(600).addClass('active-slide');
    
    currentDot.removeClass('active-dot');
    nextDot.addClass('active-dot');
});
$('.arrow-prev').click(function(){
    var currentSlide = $('.active-slide');
    var prevSlide = currentSlide.prev();
    
     var currentDot = $('.active-dot');
     var prevDot = currentDot.prev();
   
    if(prevSlide.length == 0){
        prevSlide=$('.slide').last();
        prevDot = $('.dot').last();}
     
     currentSlide.fadeOut(600).removeClass('active-slide');
    prevSlide.fadeIn(600).addClass('active-slide');
});

}; $(document).ready(main);

Answer 54acd166d3292fb064003856

2 votes

Permalink

@rose8779, The following statement lines are missing currentDot.removeClass(‘active-dot’); nextDot.addClass(‘active-dot’); in your arrow-prev -click-Event-handler

points
Submitted by Leon
over 9 years

3 comments

JC over 9 years

That doesn’t seem to work.

Leon over 9 years

@JC provide the code you are using in an -new- “Add an answer”

rose8779 about 9 years

thank you so much! @Leon

Answer 54bae69c95e378a1b70028bf

2 votes

Permalink

$(‘.arrow-prev’).click(function(){ var currentSlide = $(‘.active-slide’); var prevSlide = currentSlide.prev();

 var currentDot = $('.active-dot');
 var prevDot = currentDot.prev();

if(prevSlide.length == 0){
    prevSlide=$('.slide').last();
    prevDot = $('.dot').last();}

 currentSlide.fadeOut(600).removeClass('active-slide');
prevSlide.fadeIn(600).addClass('active-slide');

}); }; $(document).ready(main);

Change into :

$('.arrow-prev').click(function(){
    var currentSlide = $('.active-slide');
    var prevSlide = currentSlide.prev();
    
    var currentDot = $('.active-dot');
    var prevDot = currentDot.prev();
    
    
    
    if(prevSlide.length == 0) {
        prevSlide = $('.slide').last();
        prevDot = $('.dot').last();
        
        currentDot.removeClass('active-dot');
        prevDot.addClass('active-dot');
    }
   
    
    currentSlide.fadeOut(600).removeClass('active-slide');
    prevSlide.fadeIn(600).addClass('active-slide');
});    
    

} $(document).ready(main);

points
Submitted by Nikola
over 9 years

2 comments

rose8779 about 9 years

thank you!

Tara about 9 years

thank you! :)

Answer 55851d39d3292faee600062d

0 votes

Permalink

var main = function() { $(‘.dropdown-toggle’).click(function() { $(‘.dropdown-menu’).toggle(); });

$(‘.arrow-next’).click(function() { var currentSlide = $(‘.active-slide’); var nextSlide = currentSlide.next(); var currentDot = $(‘.active-dot’); var nextDot = currentDot.next();

if(nextSlide.length == 0) { nextSlide=$(‘.slide’).first(); nextDot = $(‘.dot’).first(); };

currentSlide.fadeOut(600).removeClass(‘active-slide’); nextSlide.fadeIn(600).addClass(‘active-slide’); currentDot.removeClass(‘active-dot’); nextDot.addClass(‘active-dot’); }); $(‘.arrow-prev’).click(function() { var currentSlide = $(‘.active-slide’); var prevSlide = currentSlide.prev(); var currentDot = $(‘.active-dot’); var prevDot = currentDot.prev();

if(prevSlide.length == 0) { prevSlide=$(‘.slide’).last(); prevDot = $(‘.dot’).last(); };

currentSlide.fadeOut(600).removeClass(‘active-slide’); prevSlide.fadeIn(600).addClass(‘active-slide’); currentDot.removeClass(‘active-dot’); prevDot.addClass(‘active-dot’); }); }; $(document).ready(main);

points
Submitted by javaNese
almost 9 years