nodebb-plugin-ezoic-infinite 0.5.1 → 0.5.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "0.5.1",
+  "version": "0.5.3",
   "description": "Ezoic ads with infinite scroll using a pool of placeholder IDs",
   "main": "library.js",
   "license": "MIT",

package/public/client.js

@@ -22,17 +22,16 @@ function parsePool(raw) {
   ));
 }
 
-/**
- * Harmony (topic page) selector:
- * - Posts: [component="post"]
- */
 function getTopicPosts() {
   const $p = $('[component="post"]').not('.ezoic-ad-post');
   if ($p.length) return $p;
-  // fallback
   return $('.posts .post').not('.ezoic-ad-post');
 }
 
+function isLi($el) {
+  return ($el && $el.length && (($el.prop('tagName') || '').toUpperCase() === 'LI'));
+}
+
 function removePlaceholdersByPool(pool) {
   pool.forEach(id => $('#ezoic-pub-ad-placeholder-' + id).remove());
 }
@@ -51,6 +50,37 @@ function computeWindowSlots(totalItems, interval, poolSize) {
   return out;
 }
 
+function makeBetweenWrapper($targetPost, placeholderId) {
+  if (isLi($targetPost)) {
+    return (
+      '<li class="ezoic-ad-between list-unstyled" data-ezoic-ad="1">' +
+        '<div id="ezoic-pub-ad-placeholder-' + placeholderId + '"></div>' +
+      '</li>'
+    );
+  }
+  return '<div class="ezoic-ad-between" id="ezoic-pub-ad-placeholder-' + placeholderId + '"></div>';
+}
+
+function makeAdMessageWrapper($targetPost, placeholderId) {
+  if (isLi($targetPost)) {
+    return (
+      '<li class="post ezoic-ad-post" data-ezoic-ad="1">' +
+        '<div class="content">' +
+          '<div id="ezoic-pub-ad-placeholder-' + placeholderId + '"></div>' +
+        '</div>' +
+      '</li>'
+    );
+  }
+
+  return (
+    '<div class="post ezoic-ad-post" data-ezoic-ad="1">' +
+      '<div class="content">' +
+        '<div id="ezoic-pub-ad-placeholder-' + placeholderId + '"></div>' +
+      '</div>' +
+    '</div>'
+  );
+}
+
 function insertBetweenPosts($posts, pool, interval) {
   const total = $posts.length;
   const slotsToRender = computeWindowSlots(total, interval, pool.length);
@@ -64,7 +94,7 @@ function insertBetweenPosts($posts, pool, interval) {
     const $target = $posts.eq(index);
     if (!$target.length) continue;
 
-    $target.after('<div class="ezoic-ad-between" id="ezoic-pub-ad-placeholder-' + id + '"></div>');
+    $target.after(makeBetweenWrapper($target, id));
     activeIds.push(id);
   }
   return activeIds;
@@ -83,14 +113,7 @@ function insertAdMessagesBetweenReplies($posts, pool, interval) {
     const $target = $posts.eq(index);
     if (!$target.length) continue;
 
-    const html =
-      '<div class="post ezoic-ad-post" data-ezoic-ad="1">' +
-        '<div class="content">' +
-          '<div id="ezoic-pub-ad-placeholder-' + id + '"></div>' +
-        '</div>' +
-      '</div>';
-
-    $target.after(html);
+    $target.after(makeAdMessageWrapper($target, id));
     activeIds.push(id);
   }
   return activeIds;
@@ -107,26 +130,22 @@ async function refreshAds() {
   const messageInterval = Math.max(1, parseInt(cfg.messageIntervalPosts, 10) || 3);
 
   const $posts = getTopicPosts();
-  if (!$posts.length) return; // only topic pages
+  if (!$posts.length) return;
 
-  // Clean first
   removeAdWrappers();
   removePlaceholdersByPool(betweenPool);
   removePlaceholdersByPool(messagePool);
 
   const activeIds = [];
 
-  // Between posts
   if (cfg.enableBetweenAds && betweenPool.length) {
     activeIds.push(...insertBetweenPosts($posts, betweenPool, betweenInterval));
   }
 
-  // "Ad message" between replies
   if (cfg.enableMessageAds && messagePool.length) {
     activeIds.push(...insertAdMessagesBetweenReplies($posts, messagePool, messageInterval));
   }
 
-  // Ezoic render
   if (activeIds.length && window.ezstandalone && typeof window.ezstandalone.destroyPlaceholders === 'function') {
     try { window.ezstandalone.destroyPlaceholders(); } catch (e) {}
   }