nodebb-plugin-ezoic-infinite 1.5.80 → 1.5.81

package/package.json

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

package/public/client.js

@@ -704,9 +704,10 @@ function globalGapFixInit() {
   }
 
   function pruneOrphanWraps(kindClass, items) {
-    // Message ads should remain stable (desktop + mobile). Pruning them can make it look like
-    // ads vanish on scroll and can reduce fill on long topics.
-    if (kindClass === 'ezoic-ad-message') return 0;
+    // Topic pages can be virtualized (posts removed from DOM as you scroll).
+    // When that happens, previously-inserted ad wraps may become "orphan" nodes with no
+    // nearby post containers, which leads to ads clustering together when scrolling back up.
+    // We prune only *true* orphans that are far offscreen to keep the UI stable.
     if (!items || !items.length) return 0;
     const itemSet = new Set(items);
     const wraps = document.querySelectorAll(`.${WRAP_CLASS}.${kindClass}`);
@@ -737,7 +738,10 @@ function globalGapFixInit() {
         if (wrap.getAttribute('data-ezoic-pin') === '1') return;
       } catch (e) {}
 
-      if (isFilled(wrap)) return; // never prune filled ads
+      // For message/topic pages we may prune filled or empty orphans if they are far away,
+      // otherwise consecutive "stacks" can appear when posts are virtualized.
+      const isMessage = (kindClass === 'ezoic-ad-message');
+      if (!isMessage && isFilled(wrap)) return; // never prune filled ads for non-message lists
 
       // Never prune a fresh wrap: it may fill late.
       try {
@@ -747,6 +751,20 @@ function globalGapFixInit() {
 
       if (hasNearbyItem(wrap)) return;
 
+      // For message ads: only prune if far offscreen to avoid perceived "vanishing".
+      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) {
+          // If we can't measure, be conservative.
+          return;
+        }
+      }
+
       withInternalDomChange(() => releaseWrapNode(wrap));
       removed++;
     });