fixing autosilder on ng-repeat items

[CosticaPuntaru]

adds scope.$emit('rnCarousel:CollectionUpdated', slidesCount); on repeat collection changes so other directive can listen to it
adds handle to rnCarousel:CollectionUpdated to rn-carousel-auto-slide to update the private count of slides
fixes #184

[petertellgren]

What is stopping this from getting merged in?

[revolunet]

just some time to review, rebase and merge, sorry :/ will dive into these days

[petertellgren]

great news but no need for apologies :). Just asking to see if I could help. keep up the good work

[CosticaPuntaru]

refactoring to apply review comments

[petertellgren]

Anything I can do to help speed up the merge of this PR?

[mkhatib]

Just as an FYI I've managed to fix this by moving the slideCount calculation into the increment function itself instead of globally. Not sure if this is a better solution than the one proposed in this pull just thought I'd share how I got this working.

  var increment = function () {
    var slidesCount = element.children().length;
    if (scope.carouselExposedIndex < slidesCount - 1) {
      scope.carouselExposedIndex =  scope.carouselExposedIndex + 1;
    } else {
      scope.carouselExposedIndex = 0;
    }
  };

Not sure how cheap/expensive the element.children().length to call on each increment, but this seems a very straight forward one-liner fix and worked for me.

[revolunet]

hi
i pushed an updated version that should make this easier