♻️ Opinary Ads: fix whitespace issue for AMS integration (#34650)

ampproject · Jun 2, 2021 · b5d55ee · b5d55ee
1 parent d8659df
commit b5d55ee
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/ads/_config.js b/ads/_config.js
@@ -888,7 +888,9 @@ const adConfig = jsonConfiguration({
     renderStartImplemented: true,
   },
 
-  'opinary': {},
+  'opinary': {
+    renderStartImplemented: true,
+  },
 
   'outbrain': {
     renderStartImplemented: true,

diff --git a/ads/vendors/opinary.md b/ads/vendors/opinary.md
@@ -25,7 +25,7 @@ The automated matching system is an algorithm developed by Opinary which matches
 ```html
 <amp-embed
   width="500"
-  height="500"
+  height="1"
   type="opinary"
   layout="intrinsic"
   data-client="test-success"

diff --git a/examples/amp-ad/ads.amp.html b/examples/amp-ad/ads.amp.html
@@ -1530,7 +1530,7 @@ <h2>OpenX</h2>
   </div>
 
   <h2>Opinary</h2>
-  <amp-embed width="500" height="500" type="opinary" layout="intrinsic" data-client="test-success">
+  <amp-embed width="500" height="1" type="opinary" layout="intrinsic" data-client="test-success">
   </amp-embed>
 
   <h2>Outbrain widget</h2>