- π¨π¦Canada ambient.impact Toronto
Ambient.Impact β made their first commit to this issueβs fork.
- π¨π¦Canada ambient.impact Toronto
I ran into this when integrating Hotwire Turbo into the RefreshLess module β (see π Turbo: Allow Ajax links and form submits to be handled by Drupal's Ajax framework Active ) and it took me a while to figure out that it wasn't my own code but core that wasn't correctly scoping Ajax behaviours. Did a quick commit to add the contexts. I think a lot has changed since #4, including the replacement of
jQuery.once
with the standaloneonce()
so I'm assuming the$(context).find(target)
format isn't preferred any more.Also removed the "JavaScript" tag because Drupal.org didn't let me submit with it there.
- @ambientimpact opened merge request.
- Status changed to Needs review
about 1 year ago 5:17am 31 October 2023 - π¨π¦Canada ambient.impact Toronto
This doesn't include any specific test for the context being used - should it?
- Status changed to Needs work
about 1 year ago 10:44am 31 October 2023 - π«π·France nod_ Lille
Thanks for picking this up, small suggestion on the code.
And yes, probably should have tests