search

vasyabigi / angular-slick

Angular directive for slick-carousel
http://vasyabigi.github.io/angular-slick/
MIT License
500 stars 237 forks source link

OnAfterChange does not pass slick callback paramaters #50

Open idvorkin opened 9 years ago

idvorkin commented 9 years ago

Can you pass the index and element parameters to the OnAfterChange property? I can send a pull request if that is useful.

sylvanash commented 9 years ago

I also noticed the same thing. I tried altering the code to get it to send back the index and element parameters, but I was still getting null values. At the moment, I'm using a broadcast at the onAfterChange method (not happy about it, but I needed a working solution ASAP), but I hope this will be implemented soon. Cheers

Heimingway commented 9 years ago

I've got the same problem here, both parameters are "undefined". What I need is the index directly after changing the slide. Using "currentSlideIndex" for example as a model does not work, because binding it at the "current-index" attribute laggs 1 step behind... :(

changing from slide 0 to 1 ..... $scope.currentSlideIndex is still 0.... :(

Does anyone got an idea to solve it?

idvorkin commented 9 years ago

I've fixed in my pull request: https://github.com/idvorkin/angular-slick

adskiremote commented 9 years ago

Will this be merged? Tested this works perfectly.

jimmyko commented 7 years ago

Will this be merged?

osro commented 7 years ago

Bump. This still seem to be an issue on latest version.