Corousel

[a-saran]

No description provided.

[rishichawda]

Should be Carousel. There are other spelling errors too. You can also use this extension to automatically check for any mistakes that might have accidentally crept in during typing.

[rishichawda]

We can perhaps merge the nextSlide and prevSlide to a single method that takes in the operation type and applies the modulus method of circular iteration over the items. Would reduce the need for if clause and repeated code over the methods. The formula to calculate next index should be (x+(operation)+N)%N where x is the current index, the operation is +1 or -1 and N is the total number of items.

[rishichawda]

Maybe we can take out the transition to a constant since it is never updated?

[rishichawda]

1. When auto slide/scroll is enabled, on clicking the arrow indicator - the timer/interval doesn't reset itself on the component, which makes the carousel skip the next slide or scroll twice quickly. Ideally, when clicked, it should clear the already set interval/timeout and start the timer again for the autoplay.
2. In the documentation, default prop for showArrows is mentioned as primary - but it is a boolean value with true as the default.