richardscarrott / jquery-ui-carousel

jQuery RS Carousel is a responsive and touch-enabled carousel written on top of jQuery and the jQuery UI Widget Factory providing a full and familiar API in less than 2.6kB minified and gzipped.
http://richardscarrott.github.io/jquery-ui-carousel/
192 stars 56 forks source link

animate is not defined #39

Closed niklasbulitta closed 11 years ago

niklasbulitta commented 12 years ago

Hi there,

we got an error in Line 164 - jquery.rs.carousel-touch.js: animate is not defined.

It doesn't throw the error if we just comment this line out. Is there something we missed? we can't find any definition of the animate variable.

You can reproduce the error if you activate the touch features on line 25 in your demo.html. We tested it on an ipad 1 and FireFox 11 (Win).

cheers, Nik

richardscarrott commented 12 years ago

Thanks for pointing this out, I've managed to reproduce the issue and will apply a fix when I get some time but for now, If you don't require the animate flag in the after callback, then as you've done there shouldn't be any harm in commenting it out.

richardscarrott commented 11 years ago

Only took me 11 months to get around to fixing this.