@betarena/ad-engine 0.3.0 → 0.3.2

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@betarena/ad-engine",
-	"version": "0.3.0",
+	"version": "0.3.2",
 	"private": false,
 	"description": "Betarena ad-engine widget",
 	"keywords": [
@@ -88,7 +88,7 @@
 		"vite": "5.1.6"
 	},
 	"dependencies": {
-		"@betarena/scores-lib": "4.1.0",
+		"@betarena/scores-lib": "4.14.11",
 		"@fontsource/roboto": "5.0.12",
 		"chalk": "5.3.0",
 		"colorthief": "2.6.0",

package/src/lib/Advert-Engine-Widget.svelte

@@ -106,9 +106,20 @@
      * @description
      * 📝 Condition for testing
      */
-    isStandalone = true
+    isStandalone = true,
+    /**
+     * @description
+     * 📝 Signal from the host that the current page is an article page.
+     * Zone 1 body-mounted ads (slider/popup) must only inject when this is `true`.
+     * Defaults to `false` so the widget fails closed on non-article pages.
+     */
+    isArticlePage: boolean = false
   ;
 
+  // Retained as a supported public prop for API compatibility.
+  // Device detection is UA-based; this value is not consumed internally.
+  $: void deviceWidthList;
+
   /**
    * @description
    * 📝 Component Local Interface
@@ -583,7 +594,7 @@
     // │ |:   (not from DOM-filtered targetZoneIds) so global placements inject correctly
     // │ |:   even when no zone-1 element exists in the DOM.
     // ╰─────
-    if (includesGlobalZone && (authorId || authorArticleTagIds.length > 0))
+    if (includesGlobalZone && isArticlePage && (authorId || authorArticleTagIds.length > 0))
     {
       // ╭─────
       // │ NOTE:
@@ -760,7 +771,7 @@
     // │ |: Inject STANDARD SLIDER adverts in 'document.body'
     // │ |: Guard with `includesGlobalZone` to avoid body injections on scoped refreshes.
     // ╰─────
-    else if (includesGlobalZone)
+    else if (includesGlobalZone && isArticlePage)
     {
       // [🐞]
       logger
@@ -773,11 +784,25 @@
 
       // ╭─────
       // │ NOTE:
-      // │ |: Loop over creative data and inject adverts
+      // │ |: Loop over creative data and inject adverts.
+      // │ |: Build a Set of creative IDs that belong to zone 1 so we only
+      // │ |:   inject zone-1-specific creatives rather than all slider-type creatives.
       // ╰─────
-      for (const [, adData] of mapCreative)
+      const
+        zone1CampaignIds
+          = mapZoneIdToCampaignId.get(1) ?? [],
+        zone1CreativeIds
+          = new Set
+          (
+            zone1CampaignIds
+              .flatMap(campaignId => mapCampaignIdToCreativeId.get(campaignId) ?? [])
+          )
+      ;
+
+      for (const [creativeId, adData] of mapCreative)
       {
         if (adData.type != 1) continue;
+        if (!zone1CreativeIds.has(creativeId)) continue;
 
         listAdWidgetElements.push
         (
@@ -903,7 +928,7 @@
       {
         deviceType,
         isoCountryCode: geoLocation?.country_code ?? 'EN',
-        authorId,
+        authorId: authorId ?? undefined,
         tagIds: authorArticleTagIds,
         zoneIds: targetZoneIds
       },
@@ -1062,7 +1087,7 @@
 
   {#if isStandalone}
     <link rel="preconnect" href="https://fonts.googleapis.com">
-    <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
+    <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin="anonymous">
     <link href="https://fonts.googleapis.com/css2?family=Roboto:ital,wght@0,100;0,300;0,400;0,500;0,700;0,900;1,100;1,300;1,400;1,500;1,700;1,900&display=swap" rel="stylesheet">
   {/if}
 

package/src/lib/Advert-General-Child.svelte

@@ -106,6 +106,7 @@
       width=250
       height=250
       src={adData.data.media}
+      title="Betarena advert"
     />
     <!--
       <video

package/src/lib/Advert-InterScroller-Child.svelte

@@ -262,10 +262,12 @@
     >
       <a
         target='_blank'
+        rel='noopener noreferrer'
         on:click=
         {
           () =>
           {
+            if (objectAdvertData.id === undefined) return;
             new ServiceAdEngine
             (
               betarenaEndpoint

package/src/lib/Advert-LeftSide-Child.svelte

@@ -114,11 +114,13 @@
 
 <a
   target='_blank'
+  rel='noopener noreferrer'
   href={objectAdvertData.data?.cta_link}
   on:click=
   {
     () =>
     {
+      if (objectAdvertData.id === undefined) return;
       new ServiceAdEngine
       (
         betarenaEndpoint

package/src/lib/Advert-Slide-Child.svelte

@@ -247,16 +247,11 @@
       ╰─────
       -->
       {#if isAdvertCloseBtnShown}
-        <img
-          id='close'
-          src={iconClose}
-          alt='icon-close'
-          title='icon-close'
-          loading='lazy'
-          class=
-          "
-          cursor-pointer
-          "
+        <button
+          type="button"
+          aria-label="Close advert"
+          class="cursor-pointer"
+          style="background:none;border:none;padding:0;"
           on:click=
           {
             () =>
@@ -274,7 +269,14 @@
               return;
             }
           }
-        />
+        >
+          <img
+            class='close-btn'
+            src={iconClose}
+            alt=''
+            loading='lazy'
+          />
+        </button>
       {/if}
 
       <!--
@@ -295,6 +297,7 @@
           {
             () =>
             {
+              if (adData.id === undefined) return;
               new ServiceAdEngine
               (
                 betarenaEndpoint
@@ -428,6 +431,7 @@
         {
           () =>
           {
+            if (adData.id === undefined) return;
             new ServiceAdEngine
             (
               betarenaEndpoint
@@ -520,7 +524,7 @@
 
     img
     {
-      &#close
+      &.close-btn
       {
         /* 📌 position */
         position: absolute;