drive top-scrollbar layout() from a single ResizeObserver
This commit is contained in:
@@ -115,6 +115,7 @@
|
|||||||
if (c && c.parentNode && track.nextSibling !== c) {
|
if (c && c.parentNode && track.nextSibling !== c) {
|
||||||
c.parentNode.insertBefore(track, c);
|
c.parentNode.insertBefore(track, c);
|
||||||
}
|
}
|
||||||
|
observeSizes();
|
||||||
}
|
}
|
||||||
|
|
||||||
// Board scrolled by any other means -> keep the thumb in sync. Capture-phase on
|
// Board scrolled by any other means -> keep the thumb in sync. Capture-phase on
|
||||||
@@ -185,18 +186,48 @@
|
|||||||
positionThumb(m, thumbW);
|
positionThumb(m, thumbW);
|
||||||
});
|
});
|
||||||
|
|
||||||
window.addEventListener("resize", layout);
|
// Single source of truth for recomputing the thumb: a ResizeObserver on the container
|
||||||
|
// (viewport) and #board (content). It fires exactly when the width settles/changes -- the
|
||||||
|
// right moment -- which also fixes the stale thumb from the first layout() running before the
|
||||||
|
// board finished sizing. Both are replaced on a board refresh, so re-observe the current ones
|
||||||
|
// (from reattachTrack). Without ResizeObserver, fall back to a window resize listener.
|
||||||
|
var sizeObserver = window.ResizeObserver ? new ResizeObserver(function () { layout(); }) : null;
|
||||||
|
var observedContainer = null;
|
||||||
|
var observedBoard = null;
|
||||||
|
|
||||||
// Observe the STABLE parent so we keep working after #board-container is replaced by
|
function observeSizes() {
|
||||||
// a card move / AJAX polling; re-seat the track and recompute sizes on any change.
|
if (!sizeObserver) {
|
||||||
if (window.MutationObserver) {
|
|
||||||
new MutationObserver(function () {
|
|
||||||
reattachTrack();
|
|
||||||
layout();
|
layout();
|
||||||
}).observe(boardParent, { childList: true, subtree: true });
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
layout();
|
var c = getContainer();
|
||||||
|
var b = document.getElementById("board");
|
||||||
|
|
||||||
|
if (c && c !== observedContainer) {
|
||||||
|
if (observedContainer) { sizeObserver.unobserve(observedContainer); }
|
||||||
|
sizeObserver.observe(c);
|
||||||
|
observedContainer = c;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (b && b !== observedBoard) {
|
||||||
|
if (observedBoard) { sizeObserver.unobserve(observedBoard); }
|
||||||
|
sizeObserver.observe(b);
|
||||||
|
observedBoard = b;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!sizeObserver) {
|
||||||
|
window.addEventListener("resize", layout);
|
||||||
|
}
|
||||||
|
|
||||||
|
// Re-seat the track and re-observe after #board-container is replaced by a card move / AJAX
|
||||||
|
// polling (we watch the STABLE parent, since the container itself is swapped out).
|
||||||
|
if (window.MutationObserver) {
|
||||||
|
new MutationObserver(reattachTrack).observe(boardParent, { childList: true, subtree: true });
|
||||||
|
}
|
||||||
|
|
||||||
|
observeSizes();
|
||||||
}
|
}
|
||||||
|
|
||||||
if (document.readyState === "loading") {
|
if (document.readyState === "loading") {
|
||||||
|
|||||||
Reference in New Issue
Block a user