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

'moveTo' is null or not an object (IE) #31

Closed richardwiggins closed 13 years ago

richardwiggins commented 13 years ago

Hi Matt,

Sorry to be a pain, but in IE8 (haven't tested other IE versions yet) when I click to navigate through the slides, IE throws up a JS error (below). Any ideas?

Message: 'moveTo' is null or not an object Line: 231 Char: 5 Code: 0

Cheers, Richard

Wilto commented 13 years ago

No worries, man! This has, I believe, been patched-up. I was using move as the event name, which it turns out is used by jQuery’s animate method—meaning IE was firing the move event multiple times, without a moveTo value. Have a look at the latest and let me know if we’re good to go!

richardwiggins commented 13 years ago

Howdy, I thought I was already running the latest JS but just downloaded to to make sure. Unfortunately it's still there.

Wilto commented 13 years ago

You might have to give your cache a clear—IE can be really stubborn about that.

richardwiggins commented 13 years ago

Nope, it's still there unfortunately. Tried on a different machine to make sure.

Wilto commented 13 years ago

Shenanigans. Alright sir, had to be sure—I’ll check it out ASAP!

richardwiggins commented 13 years ago

No problemo and no rush from my pov, this site is a very very long way off from going live.

Wilto commented 13 years ago

if( moveTo == null || otherwiseNotFixed ) { hat.eat(); }

Have a look at the latest on master, and we should be good to go.