Make the carousel-indicators's button more agnostic

[CyrilKrylatov]

Hello there,

I was using the carousel for my project and I was telling myself that the BUTTON elements from .carousel-indicators should have their aria-current attribute and .active class without being within a .carousel-indicators element.

In other words, for the moment,

  <div class="carousel-indicators">
    <button type="button" data-bs-target="#carouselExampleIndicators" data-bs-slide-to="0" class="active" aria-current="true" aria-label="Slide 1"></button>
    <button type="button" data-bs-target="#carouselExampleIndicators" data-bs-slide-to="1" aria-label="Slide 2"></button>
    <button type="button" data-bs-target="#carouselExampleIndicators" data-bs-slide-to="2" aria-label="Slide 3"></button>
  </div>

when I click on button[data-bs-slide-to="1"] this element will have .active class and aria-current="true" attribute.

But:

 <div class="my-custom-class-for-indicators">
   <button type="button" data-bs-target="#carouselExampleIndicators" data-bs-slide-to="0" class="active" aria-current="true" aria-label="Slide 1"></button>
   <button type="button" data-bs-target="#carouselExampleIndicators" data-bs-slide-to="1" aria-label="Slide 2"></button>
   <button type="button" data-bs-target="#carouselExampleIndicators" data-bs-slide-to="2" aria-label="Slide 3"></button>
 </div>

Will not, or for me, it should be.

What do you think of that?

Also I've never participed on Bootstrap, that could be eventually my very first PR if you think that could be a good idea.

Cheers!

✌️