- Issue created by @ndf
- π³π±Netherlands ndf Amsterdam
This bug report is based on comment #23 from mihai_brb in π Automated Drupal 10 compatibility fixes Fixed
- Status changed to Needs review
about 1 year ago 2:07pm 21 November 2023 - π³π±Netherlands ndf Amsterdam
Instead of refactoring to a new library I created the module https://www.drupal.org/project/popperjs β which contains the library
popperjs/popperjs
. - π³π±Netherlands seanB Netherlands
I would argue that popperJS was convenient while it was in core, but if that is no longer the case, we should probably move to the recommended https://floating-ui.com/.
Better yet, we might want to consider using a CSS only approach. We should try not to use JS for things that can be done using CSS.
- First commit to issue fork.
- Merge request !253403124 | Replace popper with css and remove popper dependency β (Merged) created by Matsitsjka
I attempted to replace the popper functionality with css. In order to do this I removed the visual 'arrow' from the tooltip border and I placed the tooltip in the top-left corner of the image.
- π§π·Brazil carolpettirossi Campinas - SP
Removing popperjs as suggested on MR#25 worked for us.
Attaching patch from MR#25 to use with composer.
- Status changed to RTBC
6 months ago 2:00pm 25 July 2024 - πΊπΈUnited States dpagini
I'm using the patch from #10 on my site. I actually really like the way this works, image attached. Moving to RTBC.
- π§πͺBelgium mikell
Can confirm patch in #10 fixed it for me, running D10.3
- First commit to issue fork.
-
berdir β
committed 7d9f420d on 4.x authored by
Matsitsjka β
Issue #3403124 by ndf, Matsitsjka, carolpettirossi, dpagini: Bynder...
-
berdir β
committed 7d9f420d on 4.x authored by
Matsitsjka β
Automatically closed - issue fixed for 2 weeks with no activity.