nodebb-plugin-ezoic-infinite 1.6.66 → 1.6.67

package/package.json

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

package/public/client.js

@@ -463,10 +463,8 @@
   // ─── Orphan / cluster management ────────────────────────────────────────────
   function pruneOrphanWraps(kindClass, items) {
     if (!items || !items.length) return 0;
-    const itemSet   = new Set(items);
-    const isMessage = kindClass === 'ezoic-ad-message';
-    // Between/category ads are NEVER fully released — only hidden/shown.
-    // Releasing frees IDs into the pool → re-injection at wrong positions.
+    const itemSet      = new Set(items);
+    const isMessage    = kindClass === 'ezoic-ad-message';
     const allowRelease = isMessage;
     let removed = 0;
 
@@ -490,19 +488,26 @@
 
     document.querySelectorAll('.' + WRAP_CLASS + '.' + kindClass).forEach((wrap) => {
       if (wrap.getAttribute('data-ezoic-pin') === '1') return;
-      const created = parseInt(wrap.getAttribute('data-created') || '0', 10);
-      if (created && (now() - created) < keepEmptyWrapMs()) return;
 
       if (hasNearbyItem(wrap)) {
+        // Anchor is in the DOM → always restore visibility, regardless of age.
         try { wrap.classList.remove('ez-orphan-hidden'); wrap.style.display = ''; } catch (e) {}
         return;
       }
 
-      // Anchor gone → hide.
+      // Anchor is NOT in the DOM → always hide immediately, regardless of age.
+      // KEY FIX: the previous code skipped this for fresh wraps (< keepEmptyWrapMs).
+      // That meant a wrap whose anchor topic was virtualized away within the first
+      // 60-120s would stay visible and float to the top of the list.
       try { wrap.classList.add('ez-orphan-hidden'); wrap.style.display = 'none'; } catch (e) {}
+
+      // Release (message-ads only): only after the freshness window, and only
+      // when far off-screen — we still want late-filling ads to have a chance.
       if (!allowRelease) return;
 
-      // Message-ads only: release when far off-screen.
+      const created = parseInt(wrap.getAttribute('data-created') || '0', 10);
+      if (created && (now() - created) < keepEmptyWrapMs()) return;
+
       try {
         const r  = wrap.getBoundingClientRect();
         const vh = Math.max(1, window.innerHeight || 1);