- Issue created by @Ahmad Khader
- Status changed to Needs work
5 months ago 9:02am 26 August 2024 - 🇮🇩Indonesia gausarts
Thank you.
.one()
will be executed once, best for anything executed once like AJAX content replacements, not suitable for this intention which may still be needed to be accessible at the next round.Looks like the failsafe failed here:
https://git.drupalcode.org/project/blazy/-/blob/3.0.10/js/src/components...The underlying problem may be event bubbles from child elements screwing it up. Perhaps another check is needed to wrap the entire body of the function, https://git.drupalcode.org/project/blazy/-/blob/3.0.10/js/src/components...
var transitioning = function (event) { if (event && this === event.target) { // the current body function
}
If that works, the failsafe can be removed, but not necessary for now.
- 🇯🇴Jordan Ahmad Khader
Hmmmm.
I removed the failsafe but it didn't fix the problem.I think the main issue is that
$el.off('transitionend', transitioning);
isn't removing the event for some reason. The debugger says it's removed but events trigger anyway maybe something to do with Drupal.behavior and off function. - 🇮🇩Indonesia gausarts
Yes, failsafe (the linked
setTimeout
function block) already failed, and should be ignored for now, as said.What about adding the check, wrapping the body function, as in #3?
-
gausarts →
committed a817063e on 3.0.x
Issue #3469022 by ahmad khader: Fix flybox malfunctions after being...
-
gausarts →
committed a817063e on 3.0.x
- 🇮🇩Indonesia gausarts
I haven't got time to investigate this issue. However, your proposed solution fixed the issue, while mine still failed.
Committed with minor failsafe cleanup.
Thank you for contribution.
Automatically closed - issue fixed for 2 weeks with no activity.