nodebb-plugin-ezoic-infinite 1.6.74 → 1.6.76

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.6.74",
+  "version": "1.6.76",
   "description": "Production-ready Ezoic infinite ads integration for NodeBB 4.x",
   "main": "library.js",
   "license": "MIT",

package/public/client.js

@@ -770,40 +770,42 @@ function globalGapFixInit() {
       // when orphaned, but we still avoid fully releasing them (so they can come back
       // when anchors reappear).
 
-      // Never prune a fresh wrap: it may fill late.
+      // If the wrap is anchored to current items, ensure it is visible.
+      if (hasNearbyItem(wrap)) {
+        try { wrap.classList && wrap.classList.remove('ez-orphan-hidden'); wrap.style && (wrap.style.display = ''); } catch (e) {}
+        return;
+      }
+
+      // If the anchor item is no longer in the DOM (virtualized), hide the wrap so ads never 'stack'
+      // back-to-back while scrolling. We'll restore it when its anchor comes back.
+      try { wrap.classList && wrap.classList.add('ez-orphan-hidden'); wrap.style && (wrap.style.display = 'none'); } catch (e) {}
+
+      // On topic/category lists, never release orphan wraps. Releasing moves placeholders around and makes
+      // ads appear to 'travel' from top to bottom as you scroll. Parking (hiding) keeps DOM order stable.
+      const isList = (kindClass === 'ezoic-ad-between' || kindClass === 'ezoic-ad-categories');
+      if (isList) return;
+
+      // Never release a fresh wrap: it may fill late (auction/CMP). Keep it hidden for a while.
       try {
         const created = parseInt(wrap.getAttribute('data-created') || '0', 10);
         if (created && (now() - created) < keepEmptyWrapMs()) return;
       } catch (e) {}
 
-if (hasNearbyItem(wrap)) {
-  try { wrap.classList && wrap.classList.remove('ez-orphan-hidden'); wrap.style && (wrap.style.display = ''); } catch (e) {}
-  return;
-}
-
-// If the anchor item is no longer in the DOM (virtualized), hide the wrap so ads never "stack"
-// back-to-back while scrolling. We'll recycle it when its anchor comes back.
-try { wrap.classList && wrap.classList.add('ez-orphan-hidden'); wrap.style && (wrap.style.display = 'none'); } catch (e) {}
-
-// For message ads: only release if far offscreen to avoid perceived "vanishing" during fast scroll.
-// For non-message lists (topics/categories): do not release filled wraps at all; only hide.
-if (isMessage) {
-  try {
-    const r = wrap.getBoundingClientRect();
-    const vh = Math.max(1, window.innerHeight || 1);
-    const farAbove = r.bottom < -vh * 2;
-    const farBelow = r.top > vh * 4;
-    if (!farAbove && !farBelow) return;
-  } catch (e) {
-    return;
-  }
-}
-
-// Never release filled wraps on non-message lists; hiding is enough to prevent stacks.
-if (!isMessage && isFilled(wrap)) return;
+      // For message ads: only release if far offscreen to avoid perceived 'vanishing' during fast scroll.
+      if (isMessage) {
+        try {
+          const r = wrap.getBoundingClientRect();
+          const vh = Math.max(1, window.innerHeight || 1);
+          const farAbove = r.bottom < -vh * 2;
+          const farBelow = r.top > vh * 4;
+          if (!farAbove && !farBelow) return;
+        } catch (e) {
+          return;
+        }
+      }
 
-withInternalDomChange(() => releaseWrapNode(wrap));
-removed++;
+      withInternalDomChange(() => releaseWrapNode(wrap));
+      removed++;
     });
 
     return removed;
@@ -849,6 +851,12 @@ removed++;
           const prevFilled = isFilled(prev);
           const curFilled = isFilled(w);
 
+          // On topic/category lists, prevent visible stacking by hiding the later wrap immediately.
+          if ((kindClass === 'ezoic-ad-between' || kindClass === 'ezoic-ad-categories') && prevFilled && curFilled) {
+            try { w.classList && w.classList.add('ez-stack-hidden'); w.style && (w.style.display = 'none'); } catch (e) {}
+            break;
+          }
+
           if (curFilled) {
             // If the previous one is empty (and not fresh), drop the previous instead.
             if (!prevFilled && !isFresh(prev)) {
@@ -1137,7 +1145,7 @@ function buildOrdinalMap(items) {
       if (!id) {
         // Safe mode: disable recycling for topic message ads to prevent visual "jumping"
         // (ads seemingly moving back under the first post during virtualized scroll).
-        const allowRecycle = kindClass !== 'ezoic-ad-message';
+        const allowRecycle = kindClass !== 'ezoic-ad-message' && kindClass !== 'ezoic-ad-between' && kindClass !== 'ezoic-ad-categories';
         recycledWrap = (allowRecycle && scrollDir > 0) ? pickRecyclableWrap(kindClass) : null;
         if (recycledWrap) {
           id = recycledWrap.getAttribute('data-ezoic-wrapid') || '';