nodebb-plugin-ezoic-infinite 1.5.83 → 1.5.85

package/package.json

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

package/public/client.js

@@ -1102,9 +1102,10 @@ function buildOrdinalMap(items) {
       // This avoids "Placeholder Id X has already been defined" and also ensures ads keep
       // appearing on very long infinite scroll sessions.
       if (!id) {
-        // Only recycle while scrolling down. Recycling while scrolling up tends to
-        // cause perceived instability (ads bunching/disappearing).
-        recycledWrap = scrollDir > 0 ? pickRecyclableWrap(kindClass) : null;
+        // 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 = false; // hard-disable recycling to prevent wrappers jumping to top on virtualized/infinite scroll
+        recycledWrap = (allowRecycle && scrollDir > 0) ? pickRecyclableWrap(kindClass) : null;
         if (recycledWrap) {
           id = recycledWrap.getAttribute('data-ezoic-wrapid') || '';
         }
@@ -1142,6 +1143,7 @@ function buildOrdinalMap(items) {
     let bestBottom = Infinity;
     for (const w of wraps) {
       if (!w || !w.isConnected) continue;
+      if (w.getAttribute && w.getAttribute('data-ezoic-pin') === '1') continue;
       const rect = w.getBoundingClientRect();
       if (rect.bottom < threshold) {
         if (rect.bottom < bestBottom) {
@@ -1163,17 +1165,6 @@ function buildOrdinalMap(items) {
       // Ensure minimal layout impact.
       try { wrap.style.contain = 'layout style paint'; } catch (e) {}
       try { tightenStickyIn(wrap); } catch (e) {}
-
-      const id = wrap.getAttribute('data-ezoic-wrapid');
-      if (id) {
-        setTimeout(() => {
-          try {
-            if (window.ezstandalone && typeof window.ezstandalone.showAds === 'function') {
-              window.ezstandalone.showAds(id);
-            }
-          } catch (e) {}
-        }, 0);
-      }
       return wrap;
     } catch (e) {
       return null;
@@ -1204,7 +1195,7 @@ function buildOrdinalMap(items) {
           state.allPosts,
           'curPosts'
         );
-        decluster('ezoic-ad-message');
+        // decluster disabled for stability on virtualized lists
       }
     } else if (kind === 'categoryTopics') {
       if (normalizeBool(cfg.enableBetweenAds)) {
@@ -1218,7 +1209,7 @@ function buildOrdinalMap(items) {
           state.allTopics,
           'curTopics'
         );
-        decluster('ezoic-ad-between');
+        // decluster disabled for stability on virtualized lists
       }
     } else if (kind === 'categories') {
       if (normalizeBool(cfg.enableCategoryAds)) {
@@ -1232,7 +1223,7 @@ function buildOrdinalMap(items) {
           state.allCategories,
           'curCategories'
         );
-        decluster('ezoic-ad-categories');
+        // decluster disabled for stability on virtualized lists
       }
     }