nodebb-plugin-ezoic-infinite 1.4.71 → 1.4.72

package/package.json

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

package/public/client.js

@@ -238,25 +238,17 @@
   return false;
   }
 
-  function buildWrap(target, id, kindClass, afterPos) {
-  // Use <li> when inserting inside a <ul>/<ol> list (NodeBB topic/category lists),
-  // otherwise fall back to <div>. This prevents DOM "repair" that can drop/move placeholders.
-  const parentTag = (target && target.parentElement && target.parentElement.tagName) ? target.parentElement.tagName.toUpperCase() : '';
-  const useLi = target && target.tagName && target.tagName.toUpperCase() === 'LI' && (parentTag === 'UL' || parentTag === 'OL');
-
-  const wrap = document.createElement(useLi ? 'li' : 'div');
+  function buildWrap(id, kindClass, afterPos) {
+  const wrap = document.createElement('div');
   wrap.className = `${WRAP_CLASS} ${kindClass}`;
   wrap.setAttribute('data-ezoic-after', String(afterPos));
-
-  // Ensure it behaves like a full-width block inside list layouts
   wrap.style.width = '100%';
-  if (useLi) wrap.style.listStyle = 'none';
 
   const ph = document.createElement('div');
   ph.id = `${PLACEHOLDER_PREFIX}${id}`;
   wrap.appendChild(ph);
   return wrap;
-}
+  }
 
   function findWrap(kindClass, afterPos) {
   return document.querySelector(`.${WRAP_CLASS}.${kindClass}[data-ezoic-after="${afterPos}"]`);
@@ -276,7 +268,7 @@
   insertingIds.add(id);
 
   try {
-  const wrap = buildWrap(target, id, kindClass, afterPos);
+  const wrap = buildWrap(id, kindClass, afterPos);
   target.insertAdjacentElement('afterend', wrap);
   attachFillObserver(wrap, id);
   return wrap;
@@ -290,24 +282,7 @@
   if (ids.length) destroyPlaceholderIds(ids);
   }
 
-  
-function forcePlaceholderAutoHeight(wrap, id) {
-  try {
-    if (!wrap) return;
-    const ph = wrap.querySelector && wrap.querySelector(`#${PLACEHOLDER_PREFIX}${id}`);
-    if (!ph) return;
-    ph.style.setProperty('height', 'auto', 'important');
-    ph.style.setProperty('min-height', '0px', 'important');
-    requestAnimationFrame(() => {
-      try {
-        ph.style.setProperty('height', 'auto', 'important');
-        ph.style.setProperty('min-height', '0px', 'important');
-      } catch (e) {}
-    });
-  } catch (e) {}
-}
-
-function patchShowAds() {
+  function patchShowAds() {
   const applyPatch = () => {
   try {
   window.ezstandalone = window.ezstandalone || {}, ez = window.ezstandalone;
@@ -343,12 +318,34 @@ function patchShowAds() {
   }
   }
 
-  function markFilled(wrap, id) {
+  
+  function forcePlaceholderAutoHeight(wrap, id) {
+  try {
+    if (!wrap || !id) return;
+    const ph = wrap.querySelector && wrap.querySelector(`#${PLACEHOLDER_PREFIX}${id}`);
+    if (!ph) return;
+    // Neutraliser les hauteurs réservées (inline ou CSS) qui créent un espace après la pub
+    ph.style.setProperty('height', 'auto', 'important');
+    ph.style.setProperty('min-height', '0px', 'important');
+    // Eviter le gap baseline sous les iframes/ins
+    wrap.querySelectorAll && wrap.querySelectorAll('iframe, ins').forEach(n => {
+      try { n.style.setProperty('display', 'block', 'important'); } catch (e) {}
+    });
+    requestAnimationFrame(() => {
+      try {
+        ph.style.setProperty('height', 'auto', 'important');
+        ph.style.setProperty('min-height', '0px', 'important');
+      } catch (e) {}
+    });
+  } catch (e) {}
+  }
+
+function markFilled(wrap, id) {
   try {
   if (!wrap) return;
   if (wrap.__ezoicFillObs) { wrap.__ezoicFillObs.disconnect(); wrap.__ezoicFillObs = null; }
   wrap.setAttribute('data-ezoic-filled', '1');
-  if (id) forcePlaceholderAutoHeight(wrap, id);
+  try { forcePlaceholderAutoHeight(wrap, id); } catch (e) {}
   } catch (e) {}
   }
 
@@ -427,7 +424,8 @@ function patchShowAds() {
   window.ezstandalone.cmd.push(function() {
   if (typeof window.ezstandalone.showAds === 'function') {
   // Appel batch: showAds(id1, id2, id3...)
-  window.ezstandalone.showAds(...idsArray);
+  const okIds = idsArray.filter(id => { const el = document.getElementById(`${PLACEHOLDER_PREFIX}${id}`); return el && el.isConnected; });
+  if (okIds.length) { window.ezstandalone.showAds(...okIds); }
   // Tracker tous les IDs
   idsArray.forEach(id => {
   state.lastShowById.set(id, Date.now());
@@ -576,18 +574,6 @@ function patchShowAds() {
   // Appel immédiat au lieu de 10ms delay
   callShowAdsWhenReady(id);
   }
-
-  liveArr.push({ id, wrap });
-  if (wrap && (
-  (wrap.previousElementSibling && wrap.previousElementSibling.classList && wrap.previousElementSibling.classList.contains(WRAP_CLASS)) || (wrap.nextElementSibling && wrap.nextElementSibling.classList && wrap.nextElementSibling.classList.contains(WRAP_CLASS))
-  )) {
-  try { wrap.remove(); } catch (e) {}
-  if (!(pick.recycled && pick.recycled.wrap)) {
-  try { kindPool.unshift(id); } catch (e) {}
-  usedSet.delete(id);
-  }
-  continue;
-  }
   inserted += 1;
   }
   return inserted;

package/public/style.css

@@ -1,43 +1,31 @@
-.ezoic-ad {
-  height: auto !important;
-  padding: 0 !important;
-  margin: 0 !important;
-}
-
-.ezoic-ad * {
-  margin: 0 !important;
-  padding: 0 !important;
-}
 
-/* --- Ezoic anti-blank-space fixes (NodeBB + Ajaxify) --- */
-.ezoic-ad {
-  display: none; /* shown only when filled */
-  width: 100% !important;
-  height: auto !important;
-  min-height: 0 !important;
-  padding: 0 !important;
-  margin: 0 !important;
+/* Wrapper: invisible tant que la pub n'est pas réellement insérée */
+.ezoic-ad{
+  display:none;
+  width:100%;
+  height:auto !important;
+  padding:0 !important;
+  margin:0 !important;
 }
-
 .ezoic-ad[data-ezoic-filled="1"]{
-  display: block;
+  display:block;
 }
 
-/* Ezoic placeholder should not reserve fixed height */
+/* Le placeholder ne doit jamais réserver une hauteur fixe */
 .ezoic-ad > [id^="ezoic-pub-ad-placeholder-"]{
-  height: auto !important;
-  min-height: 0 !important;
-  margin: 0 !important;
-  padding: 0 !important;
+  height:auto !important;
+  min-height:0 !important;
+  margin:0 !important;
+  padding:0 !important;
 }
 
-/* prevent baseline gaps under iframes/ins */
+/* Évite le gap baseline sous iframes/ins */
 .ezoic-ad iframe,
 .ezoic-ad ins{
-  display: block !important;
+  display:block !important;
 }
 
-/* neutralize empty spacer divs if any */
+/* Supprimer les spacers vides */
 .ezoic-ad > div:empty{
   display:none !important;
 }