Popper 2.x TODO

[XhmikosR]

• Update Popper docs links to v2 (search for https://popper.js.org/docs/ but don't blindly replace them!) Fixed by docs: Fix Popper-related docs #32390
• Update docs after Add Popper fallback behaviour #32405 Fixed by Tooltip/popover - change the default value for fallbackPlacements #32437 (Only for Tooltip/Popover)
• Update the docs for fallbackPlacements for dropdown — Popper fallback leads to worse layouting #32506 (comment) Changes were reverted in Remove custom fallbackPlacements from the dropdown #32843
• Add more JS tests to cover the changes in Update Popper to v2.x #31178

[septatrix]

#32365 should probably also be a part of this

[XhmikosR]

Not really, unless a PR is submitted.

[septatrix]

Okay, I will submit one if that is okay.

I also noticed that popper shows a warning in the console (at least when referencing the scripts manually and not using the minified bundle). Look inside the console of this pen when clicking the button. This is due to .dropdown-menu specifying a margin which poppler overwrites anyway. Do you want to track this here or create a new issue?

[XhmikosR]

Please make a new issue about the second thing. We should find a way to fix this before the stable release :)

[XhmikosR]

@rohit2sharma95 do we still need to cover the last TODO here?

[rohit2sharma95]

Yes, I see there are some missing tests. I will update the draft PR.