Allow For Custom Indicators Selector To Be Passed To Carousel Constructor

[KyleNeedham]

Currently the carousel indicators will only toggle states if you use .carousel-indicators selector, which has very specific CSS making style overrides messy.

Allowing for indicators option will make it easy to create custom indicators that keep in sync with the current active slide.

It could be as simple as:
this.$indicators = this.$element.find(options.indicators || '.carousel-indicators')

Then initialize like:
$('#my-carousel').carousel({ indicators: '.custom-indicators' })

[cvrebert]

Or instead of adding an option, perhaps we could have the JS use a class that has no styles associated with it.

[hnrch02]

As a temporary solution you could do

var carousel = $('#myCarousel').data('bs.carousel')
carousel.$indicators = carousel.$element.find('.custom-indicators')

@cvrebert Is that the plan for v4?

[KyleNeedham]

@cvrebert That's sounds like a cleaner solution, however it would require people to update their current code.

How about:
this.$indicators = this.$element.find('[data-slide-to="0"]').parent()

I know this might be the optimal solution but it wouldn't require any implementation changes.

[cvrebert]

@hnrch02 It's just my personal suggestion, but yes, it'd be for v4.

[mdo]

Adding to the v4 list.

Definitely on board with more specific classes or data attrs for JS-only behavior (saving the styles for, well, styling only).