LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

Add new list of ad configs

Browse source 
Still remain using the 5-tile banner-ad, though. There are theme and other issues with the rest.
This commit is contained in:
infinite-persistence 2022-05-26 11:09:05 +08:00
parent 816c0d9e79
commit 003992e5c6
No known key found for this signature in database
GPG key ID: B9C3252EDC3D0AA0
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
web/component/adsBanner/view.jsx
View file

@ -7,6 +7,12 @@ import useShouldShowAds from 'effects/use-should-show-ads';
const AD_SCRIPT_URL = 'https://widgets.outbrain.com/outbrain.js';
const AD_CONFIG = {
AR_18: 'AR_18', // 5 tiles.
AR_60: 'AR_60', // 6 tiles. Doesn't work well on mobile (6 tiles compresses to 1, text only).
AR_3: 'AR_3', // 4 tiles on desktop, dynamic count on mobile.
};
// ****************************************************************************
// ****************************************************************************
@ -79,7 +85,7 @@ export default function AdsBanner(props: Props) {
<div
className="banner-ad__container OUTBRAIN"
data-ob-contenturl="DROP_PERMALINK_HERE"
data-widget-id="AR_18"
data-widget-id={AD_CONFIG.AR_18}
data-ob-installation-key="ADNIMKAJDGAG4GAO6AGG6H5KP"
data-dark-mode={currentTheme === 'dark' ? 'true' : 'false'}
/>