Multiple dropdowns: Fix when inside same tag

[louismaximepiton]

Proposal to fix #36803 as a quick win. I was wondering if the data-target was an idea for v5 or v6.

Solution

Partial revert of #35752. For l.408 -> Use of the previous js to instantiate the dropdown with the great button.
Partial revert of #35748. For l.98 -> Use the previous _getMenuElement() code to have the great menu when called with the great button.

How to check the solution

Go on the input group section.
Check that the last example (the one with 2 dropdowns) display the great dropdown menu for each dropdown toggle (Action before or Action)

Things remaining

• Do the unit tests once the solution is approved

[GeoSot]

As on Line 98 you search prev & next, I suppose we should keep the same attitude here.

---

In general, I would prefer to keep the right markup solution fixing the docs and adding it in migration notes, instead of doing tricks on javascipt.
I don't know if we would count that, as a breaking change but the specific example/problem, negates the suggested markup

/cc @twbs/team

[mdo]

This PR sounds good to me if I understand it right. I get the urge for having less clever JS, but a code snippet we include in our docs no longer works after an update, so I think we should continue to support it without extra work. It's maybe less idea for us, but it keeps the developer experience better for folks building with Bootstrap :).

[GeoSot]

Ok, let's tackle it this way

@louismaximepiton please leave a comment (// todo v6 revert #37011 & change markup https://getbootstrap.com/docs/5.2/forms/input-group/ )

[GeoSot]

@louismaximepiton please add at least a test too, to cover this case

[mdo]

Thanks both! Let me know when this is good to merge :).

[louismaximepiton]

Are you fine with the added tests cases ?

[GeoSot]

You are more than great @louismaximepiton . Thank you a lot 🚀