nodebb-plugin-ezoic-infinite 1.5.92 → 1.5.94

package/package.json

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

package/public/client.js

@@ -1,17 +1,6 @@
 (function () {
   'use strict';
 
-function ezoicTrySetAnchor(wrap, el, kindClass, afterPos) {
-  try {
-    if (!wrap || !el) return;
-    var pid = el.getAttribute && (el.getAttribute('data-pid') || el.getAttribute('data-id'));
-    var tid = el.getAttribute && (el.getAttribute('data-tid') || el.getAttribute('data-topic-id'));
-    var key = kindClass + ':' + (pid ? ('pid:' + pid) : (tid ? ('tid:' + tid) : ('pos:' + String(afterPos || 0))));
-    wrap.setAttribute('data-ezoic-anchor', key);
-  } catch (e) {}
-}
-
-
   // Track scroll direction to avoid aggressive recycling when the user scrolls upward.
   // Recycling while scrolling up is a common cause of ads "bunching" and a "disappearing too fast" feeling.
   let lastScrollY = 0;
@@ -651,7 +640,6 @@ function globalGapFixInit() {
     const wrap = document.createElement('div');
     wrap.className = `${WRAP_CLASS} ${kindClass}`;
     wrap.setAttribute('data-ezoic-after', String(afterPos));
-      ezoicTrySetAnchor(wrap, el, kindClass, afterPos);
     wrap.setAttribute('data-ezoic-wrapid', String(id));
     wrap.setAttribute('data-created', String(now()));
     // "Pinned" placements (after the first item) should remain stable.
@@ -716,143 +704,12 @@ function globalGapFixInit() {
   }
 
   function pruneOrphanWraps(kindClass, items) {
-    // Topic pages can be virtualized (posts removed from DOM as you scroll).
-    // When that happens, previously-inserted ad wraps may become "orphan" nodes with no
-    // nearby post containers, which leads to ads clustering together when scrolling back up.
-    // We prune only *true* orphans that are far offscreen to keep the UI stable.
-    if (!items || !items.length) return 0;
-    const itemSet = new Set(items);
-    const wraps = document.querySelectorAll(`.${WRAP_CLASS}.${kindClass}`);
-    let removed = 0;
-
-    const isFilled = (wrap) => {
-      return !!(wrap.querySelector('iframe, ins, img, video, [data-google-container-id]'));
-    };
-
-    const hasNearbyItem = (wrap) => {
-      // NodeBB/skins can inject separators/spacers; be tolerant.
-      let prev = wrap.previousElementSibling;
-      for (let i = 0; i < 14 && prev; i++) {
-        if (itemSet.has(prev)) return true;
-        prev = prev.previousElementSibling;
-      }
-      let next = wrap.nextElementSibling;
-      for (let i = 0; i < 14 && next; i++) {
-        if (itemSet.has(next)) return true;
-        next = next.nextElementSibling;
-      }
-      return false;
-    };
-
-    wraps.forEach((wrap) => {
-      // Never prune pinned placements.
-      try {
-        if (wrap.getAttribute('data-ezoic-pin') === '1') return;
-      } catch (e) {}
-
-      // 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
-
-      // Never prune a fresh wrap: it may fill late.
-      try {
-        const created = parseInt(wrap.getAttribute('data-created') || '0', 10);
-        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 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) {}
-
-// For message ads: only release if far offscreen to avoid perceived "vanishing" during fast scroll.
-if (isMessage) {
-  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;
-  }
-}
-
-withInternalDomChange(() => releaseWrapNode(wrap));
-removed++;
-    });
-
-    return removed;
-  }
-
   function decluster(kindClass) {
-    // Remove "near-consecutive" wraps (keep the first). Be tolerant of spacer nodes.
-    const wraps = Array.from(document.querySelectorAll(`.${WRAP_CLASS}.${kindClass}`));
-    if (wraps.length < 2) return 0;
-
-    const isWrap = (el) => !!(el && el.classList && el.classList.contains(WRAP_CLASS));
-
-    const isFilled = (wrap) => {
-      return !!(wrap && wrap.querySelector && wrap.querySelector('iframe, ins, img, video, [data-google-container-id]'));
-    };
-
-    const isFresh = (wrap) => {
-      try {
-        const created = parseInt(wrap.getAttribute('data-created') || '0', 10);
-        return created && (now() - created) < keepEmptyWrapMs();
-      } catch (e) {
-        return false;
-      }
-    };
-
-    let removed = 0;
-    for (const w of wraps) {
-      // Never decluster pinned placements.
-      try {
-        if (w.getAttribute && w.getAttribute('data-ezoic-pin') === '1') continue;
-      } catch (e) {}
-
-      let prev = w.previousElementSibling;
-      for (let i = 0; i < 3 && prev; i++) {
-        if (isWrap(prev)) {
-          // If the previous wrap is pinned, keep this one (spacing is intentional).
-          try {
-            if (prev.getAttribute && prev.getAttribute('data-ezoic-pin') === '1') break;
-          } catch (e) {}
-
-          // 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 (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++;
-          }
-          break;
-        }
-        prev = prev.previousElementSibling;
-      }
-    }
-    return removed;
-  }
+  return;
+}
 
   // ---------------- show (preload / fast fill) ----------------