Skip to content
GitLab

Fix proposal for Tooltips

Merged Louis-Maxime Piton requested to merge github/fork/louismaximepiton/main-lmp-tooltip-fix into main

Closes #36069 (closed)

Solutions

Thanks to everyone that contributed to the issue, here are the different solutions that came to my mind :

  • @atomiks : Change the CSS to have the expected behavior. (Which is actually implemented in the PR)
  • @GeoSot : Change the js file to have a different behavior which contains some bugs in rare cases imo. (Which modifies the PopperConfig modifier)

I tried for this to modify https://github.com/twbs/bootstrap/blob/51535cd95ac8eb5746e19c057aeabdbcafef3a8b/js/src/tooltip.js#L425 in order to have :

fallbackPlacements: (this._config.placement === 'top' || this._config.placement === 'bottom') ? ['top', 'bottom', 'right', 'left'] : ['right', 'left', 'top', 'bottom']

As the CSS solution seems to do the trick, I didn't explored too much in this way, and the bug remains (just moved away), but I could try to if needed !

Thanks again for the contributors of the issue ❤️