Carousel: No listeners for mobile carousels with `data-bs-ride=boolean`

Prerequisites

• I have searched for duplicate or closed issues
• I have validated any HTML to avoid common problems
• I have read the contributing guidelines

Describe the issue

Seems to be there for quite a long time.

The main issue is that if data-bs-ride is set to anything but carousel, the constructor() isn’t called (whereas there’s a check inside to avoid launch cycle()). This leads to unset (at least) touch listeners which, in mobile views, is kinda weird IMO.

I don’t have the time to test it atm, but I think there’s only a change on the selector would do the trick ?

Reduced test cases

Go on https://twbs-bootstrap.netlify.app/docs/5.2/components/carousel/#with-controls (which works well with mobile listeners) Go on https://twbs-bootstrap.netlify.app/docs/5.2/components/carousel/#with-indicators (which is broken with mobile listeners until we click one button)

What operating system(s) are you seeing the problem on?

Windows

What browser(s) are you seeing the problem on?

Chrome, Firefox

What version of Bootstrap are you using?

5.2.2 (current_main_branch)

[patrickhlauke]

The solution is real simple if we are sure we want this attitude 😃 #37295 (comment)

i think it’s even simpler than that: no need to check if there’s a data-bs-ride attribute at all, just attach the touch listeners on initialisation, all the time, just like with mouse listeners. touch functionality is not dependent on whether it autoscrolls or scrolls after first interaction.

[patrickhlauke]

However, @louismaximepiton pointed something here. The carousel touch listeners are registered after the initialization, in contradiction with click event listeners that are registered on document

ah yes, event listeners should all register at the same time at initialisation, not just after first interaction