tags:

views:

107

answers:

2

I've written this simple Carousel. At the moment I'm using setInterval to run my nextSlide function at certain intervals. I want to defer the timer from running when a user clicks on the navigation links for a certain length of time.

Check it out here http://jsbin.com/uzixi3/3/edit

Any feedback on how the rest is written would be good too.

+1  A: 

You can save the return value of setInterval in a variable to refer to it later - that way you can cancel it if you need to, or restart it.

See this MDC article for more details.

The basics are:

intervalID = setInterval(flashText, 1000);

//do something...

clearInterval(intervalID);
Skilldrick
Thanks for your help
Reynish
+2  A: 

You could do something like this: http://jsbin.com/uzixi3/5/edit

The interval part is here:

var int = setInterval($.fn.nextSlide, 3000);
$("#slideNavigation a").click(function() {
  clearInterval(int);
  setTimeout(function() {
    setInterval($.fn.nextSlide, 3000);
  }, 10000);
});

I made some other tweaks as well though, for example you can use a switch statement to make .nextSlide() much more readable and cheaper.

Overall though, there's no reason to make these functions as extension methods on jjquery itself since they don't interact with objects, they can just be methods scoped to the closure like this: http://jsbin.com/uzixi3/6/edit

If the methods were actually running on $('#slideContainer'), e.g. $('#slideContainer').nextSlide() and inside your methods you used this.animate() and this.css() it might make a bit more sense, just some thoughts that may help you get more flexible as you go.

Nick Craver
Cheers Nick. Thanks for the advice, most helpful
Reynish
@Reynish - welcome :)
Nick Craver
hihi, had much to fun playing with the sliders to answer in time ;-)
Morten Bergfall