fix: drop hx-trigger once — rely on browser cache for dedup

This commit is contained in:
Pontoporeia
2026-04-24 13:27:47 +02:00
parent 743268cf1b
commit 769beae4ee

View File

@@ -21,34 +21,18 @@
popover.style.top = top + 'px'; popover.style.top = top + 'px';
} }
// After htmx swaps content in, show and position the popover
document.body.addEventListener('htmx:afterSwap', function (e) {
if (e.detail.target !== popover) return;
popover.hidden = false;
if (currentAnchor) position(currentAnchor);
});
// On subsequent hovers (already cached by browser / htmx once),
// just reposition and show — htmx won't re-fire due to `once`,
// so we handle show/position on mouseenter ourselves.
document.body.addEventListener('mouseenter', function (e) { document.body.addEventListener('mouseenter', function (e) {
var a = e.target.closest('[data-student-name]'); var a = e.target.closest('[data-student-name]');
if (!a) return; if (!a) return;
currentAnchor = a; currentAnchor = a;
// If popover already has this student's content, just show it
if (popover.dataset.loadedFor === a.dataset.studentName) {
position(a);
popover.hidden = false;
}
}, true); }, true);
// Mark which student is loaded after swap
document.body.addEventListener('htmx:afterSwap', function (e) { document.body.addEventListener('htmx:afterSwap', function (e) {
if (e.detail.target !== popover || !currentAnchor) return; if (e.detail.target !== popover) return;
popover.dataset.loadedFor = currentAnchor.dataset.studentName; if (currentAnchor) position(currentAnchor);
popover.hidden = false;
}); });
// Hide on mouse leave (both anchor and popover)
document.body.addEventListener('mouseleave', function (e) { document.body.addEventListener('mouseleave', function (e) {
if (!e.target.closest('[data-student-name]') && !e.target.closest('#student-popover')) return; if (!e.target.closest('[data-student-name]') && !e.target.closest('#student-popover')) return;
setTimeout(function () { setTimeout(function () {