nodebb-plugin-ezoic-infinite 1.5.78 → 1.5.79

package/package.json

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

package/public/client.js

@@ -22,9 +22,11 @@
 
   // Preload margins
   const PRELOAD_MARGIN_DESKTOP = '2600px 0px 2600px 0px';
-  const PRELOAD_MARGIN_MOBILE = '1500px 0px 1500px 0px';
-  const PRELOAD_MARGIN_DESKTOP_BOOST = '4200px 0px 4200px 0px';
-  const PRELOAD_MARGIN_MOBILE_BOOST = '2500px 0px 2500px 0px';
+  // Mobile: larger preload window so ad fill requests start earlier and
+  // users don't scroll past empty placeholders.
+  const PRELOAD_MARGIN_MOBILE = '3200px 0px 3200px 0px';
+  const PRELOAD_MARGIN_DESKTOP_BOOST = '5200px 0px 5200px 0px';
+  const PRELOAD_MARGIN_MOBILE_BOOST = '5200px 0px 5200px 0px';
 
   const BOOST_DURATION_MS = 2500;
   const BOOST_SPEED_PX_PER_MS = 2.2; // ~2200px/s
@@ -751,11 +753,24 @@ function globalGapFixInit() {
             if (prev.getAttribute && prev.getAttribute('data-ezoic-pin') === '1') break;
           } catch (e) {}
 
-          // Don't decluster two "fresh" empty wraps — it can reduce fill on slow auctions.
-          // Only decluster when at least one is filled, or when the newer one is stale.
+          // Never remove a wrap that is already filled; otherwise it looks like
+          // ads "disappear" while scrolling. Only remove the empty neighbour.
           const prevFilled = isFilled(prev);
           const curFilled = isFilled(w);
-          if (prevFilled || curFilled || !isFresh(w)) {
+
+          if (curFilled) {
+            // If the previous one is empty (and not fresh), drop the previous instead.
+            if (!prevFilled && !isFresh(prev)) {
+              withInternalDomChange(() => releaseWrapNode(prev));
+              removed++;
+            }
+            break;
+          }
+
+          // Current is empty.
+          // Don't decluster two "fresh" empty wraps — it can reduce fill on slow auctions.
+          // Only decluster when previous is filled, or when current is stale.
+          if (prevFilled || !isFresh(w)) {
             withInternalDomChange(() => releaseWrapNode(w));
             removed++;
           }
@@ -817,10 +832,16 @@ function globalGapFixInit() {
     try { io && io.observe(ph); } catch (e) {}
 
     // If already near viewport, fire immediately.
+    // Mobile tends to scroll faster + has slower auctions, so we fire earlier.
     try {
       const r = ph.getBoundingClientRect();
-      const screens = isBoosted() ? 5.0 : 3.0;
-      const minBottom = isBoosted() ? -1500 : -800;
+      const mobile = isMobile();
+      const screens = isBoosted()
+        ? (mobile ? 9.0 : 5.0)
+        : (mobile ? 6.0 : 3.0);
+      const minBottom = isBoosted()
+        ? (mobile ? -2600 : -1500)
+        : (mobile ? -1400 : -800);
       if (r.top < window.innerHeight * screens && r.bottom > minBottom) enqueueShow(id);
     } catch (e) {}
   }