nodebb-plugin-ezoic-infinite 1.6.73 → 1.6.75

package/package.json

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

package/public/client.js

@@ -220,6 +220,42 @@ function tightenEzoicMinHeight(wrap) {
   } catch (e) {}
 }
 
+// If a placement never fills, Ezoic may still leave an inline `min-height:400px !important`
+// on nested containers, creating large "blank" blocks. On SPA/infinite scroll, some
+// auctions can fail (network/CMP/adblock). We collapse those gaps after a grace period.
+function collapseEmptyGap(wrap) {
+  try {
+    if (!wrap || !wrap.isConnected) return;
+    if (isFilledNode(wrap)) return;
+
+    const created = parseInt(wrap.getAttribute('data-created') || '0', 10);
+    if (!created) return;
+    // Give plenty of time for late fill (CMP / header bidding).
+    if ((now() - created) < 12000) return;
+
+    // Find the nested node that carries the 400px min-height.
+    const nodes = wrap.querySelectorAll('.ezoic-ad, .ezoic-ad-adaptive');
+    nodes.forEach((n) => {
+      const st = (n.getAttribute('style') || '').toLowerCase();
+      if (st.includes('min-height:400')) {
+        try { n.style.setProperty('min-height', '1px', 'important'); } catch (e) { n.style.minHeight = '1px'; }
+        try { n.style.setProperty('height', 'auto', 'important'); } catch (e) {}
+      }
+    });
+
+    // Also collapse our wrapper's reserved height.
+    try { wrap.style.setProperty('min-height', '1px', 'important'); } catch (e) { wrap.style.minHeight = '1px'; }
+    try { wrap.style.setProperty('height', 'auto', 'important'); } catch (e) {}
+  } catch (e) {}
+}
+
+function collapseEmptyGaps(kindClass) {
+  try {
+    const wraps = document.querySelectorAll(`.${WRAP_CLASS}.${kindClass}`);
+    wraps.forEach((w) => collapseEmptyGap(w));
+  } catch (e) {}
+}
+
 function watchWrapForFill(wrap) {
   try {
     if (!wrap || wrap.__ezFillObs) return;
@@ -738,17 +774,16 @@ function globalGapFixInit() {
         if (wrap.getAttribute('data-ezoic-pin') === '1') return;
       } catch (e) {}
 
-      // For message pages AND category topic lists, NodeBB may virtualize/recycle items.
-      // If we never hide/prune filled wraps, they can "pile up" when their anchor items
-      // temporarily leave the DOM (especially on /category/... lists).
+      // Virtualized lists can remove anchors from the DOM while keeping our wraps.
+      // If we keep *filled* wraps alive when their anchors disappear, they will visually
+      // "stack" (back-to-back) when the user scrolls back up.
       //
-      // Strategy:
-      //  - always hide orphan wraps (filled or empty) when their anchor item is gone
-      //  - only *release* (remove) them if they are far offscreen, to avoid "vanishing" feel
+      // Topic list pages (categoryTopics) are also virtualized in some themes/plugins,
+      // so we must treat between-ads similarly to message-ads.
       const isMessage = (kindClass === 'ezoic-ad-message');
-      const isTopicList = (kindClass === 'ezoic-ad-between');
-      const allowOrphanForFilled = (isMessage || isTopicList);
-      if (!allowOrphanForFilled && isFilled(wrap)) return; // keep filled ads for non-virtualized lists
+      const isBetween = (kindClass === 'ezoic-ad-between');
+      const isVirtualizedList = isMessage || isBetween;
+      if (!isVirtualizedList && isFilled(wrap)) return; // keep filled ads for non-virtualized lists
 
       // Never prune a fresh wrap: it may fill late.
       try {
@@ -756,30 +791,34 @@ function globalGapFixInit() {
         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 (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) {}
+      // If the anchor item is no longer in the DOM (virtualized), hide the wrap so ads never "stack"
+      // back-to-back while scrolling. When the anchor comes back, the wrap can be shown again.
+      try {
+        wrap.classList && wrap.classList.add('ez-orphan-hidden');
+        wrap.style && (wrap.style.display = 'none');
+      } catch (e) {}
 
-// For message ads and topic-list ads: only release if far offscreen to avoid perceived "vanishing" during fast scroll.
-if (isMessage || isTopicList) {
-  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;
-  }
-}
+      // Release only if far offscreen (keep UX stable). Use looser thresholds for topic lists.
+      try {
+        const r = wrap.getBoundingClientRect();
+        const vh = Math.max(1, window.innerHeight || 1);
+        const farAbove = r.bottom < -vh * (isBetween ? 4 : 2);
+        const farBelow = r.top > vh * (isBetween ? 8 : 4);
+        if (!farAbove && !farBelow) return;
+      } catch (e) {
+        return;
+      }
 
-withInternalDomChange(() => releaseWrapNode(wrap));
-removed++;
+      withInternalDomChange(() => releaseWrapNode(wrap));
+      removed++;
     });
 
     return removed;
@@ -805,6 +844,7 @@ removed++;
       }
     };
 
+    const lookback = (kindClass === 'ezoic-ad-between') ? 12 : 3;
     let removed = 0;
     for (const w of wraps) {
       // Never decluster pinned placements.
@@ -813,7 +853,7 @@ removed++;
       } catch (e) {}
 
       let prev = w.previousElementSibling;
-      for (let i = 0; i < 3 && prev; i++) {
+      for (let i = 0; i < lookback && prev; i++) {
         if (isWrap(prev)) {
           // If the previous wrap is pinned, keep this one (spacing is intentional).
           try {
@@ -1203,6 +1243,7 @@ function buildOrdinalMap(items) {
           'curPosts'
         );
         decluster('ezoic-ad-message');
+        collapseEmptyGaps('ezoic-ad-message');
       }
     } else if (kind === 'categoryTopics') {
       if (normalizeBool(cfg.enableBetweenAds)) {
@@ -1217,6 +1258,7 @@ function buildOrdinalMap(items) {
           'curTopics'
         );
         decluster('ezoic-ad-between');
+        collapseEmptyGaps('ezoic-ad-between');
       }
     } else if (kind === 'categories') {
       if (normalizeBool(cfg.enableCategoryAds)) {
@@ -1231,6 +1273,7 @@ function buildOrdinalMap(items) {
           'curCategories'
         );
         decluster('ezoic-ad-categories');
+        collapseEmptyGaps('ezoic-ad-categories');
       }
     }