nodebb-plugin-ezoic-infinite 1.6.72 → 1.6.74

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.6.72",
+  "version": "1.6.74",
   "description": "Production-ready Ezoic infinite ads integration for NodeBB 4.x",
   "main": "library.js",
   "license": "MIT",
@@ -18,4 +18,4 @@
     "compatibility": "^4.0.0"
   },
   "private": false
-}
+}

package/public/client.js

@@ -133,7 +133,29 @@ function tightenEzoicMinHeight(wrap) {
     if (!wrap || !wrap.querySelector) return;
 
     const iframes = wrap.querySelectorAll('iframe');
-    if (!iframes || !iframes.length) return;
+
+    // If the wrap is still empty, Ezoic often leaves inline `min-height:400px !important` on
+    // nested `.ezoic-ad` containers, which creates large blank gaps. We can't remove it with CSS.
+    // We collapse those gaps after a short grace period, while still allowing late fill.
+    if (!iframes || !iframes.length) {
+      try {
+        const created = parseInt(wrap.getAttribute('data-created') || '0', 10);
+        const age = created ? (now() - created) : 0;
+        // Give auctions/CMP a bit of time; then collapse the notorious 400px gap.
+        if (age >= 12000) {
+          const nodes = wrap.querySelectorAll('.ezoic-ad, .ezoic-ad-adaptive');
+          nodes.forEach((n) => {
+            const st = (n.getAttribute('style') || '').toLowerCase();
+            if (st.includes('min-height:400') || st.includes('min-height: 400') || st.includes('min-height:400px')) {
+              try { n.style.setProperty('min-height', '1px', 'important'); } catch (e) { n.style.minHeight = '1px'; }
+              try { n.style.setProperty('height', 'auto', 'important'); } catch (e) {}
+              try { n.style.setProperty('line-height', '0', 'important'); } catch (e) {}
+            }
+          });
+        }
+      } catch (e) {}
+      return;
+    }
 
     // Find the closest "big" ezoic container that carries the 400px min-height.
     const firstIframe = iframes[0];
@@ -741,7 +763,12 @@ function globalGapFixInit() {
       // 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
+      // IMPORTANT:
+      // On category/topic lists, NodeBB can virtualize/recycle list items while scrolling.
+      // If we keep *filled* wraps when their anchor items are gone, they can visually
+      // "stack" together on upward scroll. We therefore allow filled wraps to be hidden
+      // 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.
       try {
@@ -759,6 +786,7 @@ if (hasNearbyItem(wrap)) {
 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();
@@ -771,6 +799,9 @@ if (isMessage) {
   }
 }
 
+// Never release filled wraps on non-message lists; hiding is enough to prevent stacks.
+if (!isMessage && isFilled(wrap)) return;
+
 withInternalDomChange(() => releaseWrapNode(wrap));
 removed++;
     });
@@ -1019,6 +1050,8 @@ removed++;
               watchWrapForFill(ww);
               setTimeout(() => { try { tightenEzoicMinHeight(ww); } catch (e) {} }, 900);
               setTimeout(() => { try { tightenEzoicMinHeight(ww); } catch (e) {} }, 2200);
+              // Collapse the notorious 400px inline min-height if the slot stays empty.
+              setTimeout(() => { try { tightenEzoicMinHeight(ww); } catch (e) {} }, 12500);
             }
           } catch (e) {}
           setTimeout(() => { clearTimeout(hardTimer); release(); }, 650);
@@ -1091,21 +1124,6 @@ function buildOrdinalMap(items) {
       const el = ordinalMap.get(afterPos);
       if (!el) continue;
       if (!el || !el.isConnected) continue;
-
-      // Viewport guard (loose): NodeBB may load batches above the fold when the user scrolls up.
-      // If we inject immediately into anchors that are *far* above the viewport, wraps tend to
-      // accumulate/pile near the top once the DOM is recycled.
-      //
-      // Important: do NOT skip when the anchor is only slightly above the viewport (e.g. -10px),
-      // otherwise injection feels "missing" compared to v17. We only skip when it's well above.
-      try {
-        if (scrollDir < 0) {
-          const bottom = el.getBoundingClientRect().bottom;
-          const cutoff = -Math.max(240, (window.innerHeight || 800) * 0.75);
-          if (bottom < cutoff) continue;
-        }
-      } catch (e) {}
-
       if (isAdjacentAd(el)) continue;
       if (findWrap(kindClass, afterPos)) continue;