- Issue created by @laetitia_al
- 🇫🇷France Grimreaper France 🇫🇷
When clicking on the contextual links pen, it opens and you can scroll down.
This is a Core feature/behavior, nothing special done in Sobki.
I guess in Core, there should be a check in JS to open to the top instead of to the bottom, but would require to use an external library that handles that.
- 🇫🇷France Grimreaper France 🇫🇷
Or maybe in USB we can override contextual links to use https://getbootstrap.com/docs/5.3/components/popovers/ to render it. And the direction should be automatic.
- 🇫🇷France Grimreaper France 🇫🇷
Need help with contextual link JS, how better to integrate.
- Merge request !266Issue #3503645 by grimreaper, laetitia_al: Contextual links: use popover → (Merged) created by Grimreaper
- 🇫🇷France Grimreaper France 🇫🇷
2 remaining problems:
- popover state is not synced with the backbone model state
- not possible to access the links inside the popover with the keyboard - 🇫🇷France Grimreaper France 🇫🇷
Morning thought, instead of popover, maybe change into a dropdown (which if I remember correctly can also open automatically where there is enough space).
- 🇫🇷France Grimreaper France 🇫🇷
discussed with @tocab, Bootstrap popovers can't have interactive links. So focus don't go in it.
I will try to use dropdowns instead.
-
grimreaper →
committed b743313c on 5.1.x
Issue #3503645 by grimreaper, laetitia_al, tocab: Contextual links: use...
-
grimreaper →
committed b743313c on 5.1.x
-
grimreaper →
committed 04a81363 on 5.1.x
Revert "Issue #3503645 by grimreaper, laetitia_al, tocab: Contextual...
-
grimreaper →
committed 04a81363 on 5.1.x
- 🇫🇷France Grimreaper France 🇫🇷
Change reverted and marking as postponed until Core issue is resolved.