search

Wilto / Dynamic-Carousel

A carousel plugin built for responsive layouts. There are many like it, but this one is mine.
matmarquis.com/carousel/
472 stars 112 forks source link

addNav controls not working in IE7+error warning #21

Closed chillfinn closed 13 years ago

chillfinn commented 13 years ago

The generated nav controls for prev/next don't seem to work in IE7

Also I'm getting an error message stating: 'moveTo' is null or not an object on line 245

Yeh IE7 sucks but...

Wilto commented 13 years ago

But, no reason we shouldn’t be able to make it work anyway, yeah? I spent some time looking into this today, and it seems like a weird one—a fix is in the works, though.

chillfinn commented 13 years ago

Yeh, we gotta cover all bases as pro's :D Otherwise you've done a brilliant job there, thanks!

Wilto commented 13 years ago

That was, in fact, a weird one—I didn’t realize that the “move” event was overlapping an event built into jQuery’s animate() functionality, and that overlap was firing the event a bunch of times. Turned up an unrelated issue in the navState function and fixed that as well. Looks like we’re back to 100% in 6/7/8.

chillfinn commented 13 years ago

I'm still getting the same issue - is that committed to the download?