fix(carousel): respect the order of the slides

[ayakovlevgh]

I've added a new parameter (index) to the slide directive which is used to order slides:

<slide ng-repeat="slide in slides" index="$index">
    <!-- slide content here -->
</slide>

You can see it on plunker:

• carousel without these changes
• carousel with changes

Fixes #488

[wesleycho]

Can you extend this so that it can support functions as well? It should support all of the possibilities of orderBy IMO.

[ayakovlevgh]

Sorry, but I'm not sure that I understand what you mean. In this peace of code we're just ordering the carousel indicators by the slide index parameter. Do you mean that we should do something like this:

<li ng-repeat="slide in slides | orderBy:orderIndicators track by $index" ng-class="{active: isActive(slide)}" ng-click="select(slide)"></li>

and then in controller:

$scope.orderIndicators = function(slide) {
    return slide.index;
};

What's the purpose of such extension?

[wesleycho]

So my concern is if the user is doing something like

<slide ng-repeat="slide in slides | orderBy: customOrderFn">
...
</slide>

I'm not sure whether that custom order would be respected by this implementation.

[ayakovlevgh]

it would be, please check this plunker: http://plnkr.co/edit/pmCFIH0GCpBjNBWY8g1D?p=preview.

[wesleycho]

Landed as b5f220f, thanks for the work here!