I have a for loop that is iterating through some data. Every 10th iteration I need to display an ad from Google Adsense.
However, when I insert the Adsense code the rest of my code breaks -- I get an "Unexpected EOF."
Here is what I have, based off of this example from Google:
for (var i = 0; i < json.length; i++) {
if ((i % 10) == 0)$("#contentRow").append(`
<script async src="//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js">
</script>
<ins class="adsbygoogle" style="display:block"
data-ad-format="fluid"
data-ad-client="ca-pub-1234567891234567"
data-ad-slot="1234567890"></ins>
<script>
(adsbygoogle = window.adsbygoogle || []).push({});
</script>
`);
$("#contentRow").append(`-- iterating my data here... --`)
This all works fine until I add the Adsense code. Replacing the Adsense code with hello world, for example, works just fine. But once the Adsense code is included everything breaks, think because of the script tags. Any help?
Per https://stackoverflow.com/a/22090482/697079 you need to escape your script tag(s).
<\/script>
I'm receiving 404's assuming because of the account settings. https://jsfiddle.net/zem4xqsw/
I built a WordPress site (Newspaper template) and wanted to add Google AdSense. After approval I added the code, but it gives me the error listed below:
Uncaught
M {message: "adsbygoogle.push() error: Only one 'enable_page_level_ads' allowed per page.", name: "TagError", pbr: true, stack: "TagError: adsbygoogle.push() error: Only one 'enab…syndication.com/pagead/js/adsbygoogle.js:1:21800)"}
message: "adsbygoogle.push() error: Only one 'enable_page_level_ads' allowed per page."
I have no idea what to do. Does anybody have any idea?
hmm.. maybe script code that Google Adsense had provided was written down two or more on your website.
<script async src="https://pagead2.googlesyndication.com/pagead/js/adsbygoogle.js"></script>
<script>
(adsbygoogle = window.adsbygoogle || []).push({
google_ad_client: "your client id",
enable_page_level_ads: true
});
</script>
check this script if it was duplicated.
If you clear the duplicate, the error will disappear.
Have a nice day :D
Check your website's page source for multiple AdSense code. Use Ctrl+U (or view-source:website.com), Ctrl+F and search for pub-.
If you are using WordPress you might have two code
Inserted via Google Sitekit plugin
(adsbygoogle = window.adsbygoogle || []).push({
google_ad_client: "ca-pub-****************",
enable_page_level_ads: true,
tag_partner: "site_kit"
});
</script>
Direct insertion
<script data-ad-client="ca-pub-****************" async src="https://pagead2.googlesyndication.com/pagead/js/adsbygoogle.js"></script>
Removing one of these code to resolve this issue.
I have a problem with the banners of Adsense.
I have a page containing 3 banners with the fixed size 336 x 280. On this page, i see one or more blank ads in a random way. I explain better:
Sometimes I can see only one banner working and the other two are blank, sometimes two banners are working and the other is blank, sometimes none of the banners is working and so on...
The code is the same for all the banners:
<div id="banner_rectangle" style="width:336px;height:280px">
<script async src="//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js"></script>
<!-- 336_280_text_images -->
<ins class="adsbygoogle"
style="display:inline-block;width:336px;height:280px"
data-ad-client="ca-pub-XXXXXXXXXXXXXXX"
data-ad-slot="XXXXXXXXXX"></ins>
<script>
(adsbygoogle = window.adsbygoogle || []).push({});
</script></div>
As you can see in the code of Google there is already a directive for the size of the banner, and there is also the parent div named banner_rectangle with the same directive.
How can I fix this problem?
I will advise you place all script tags sequentially based on how dependent they are and secondly place all your script tag just before your body tag.
Something like
<script ...
<script src="..."></script>
</body>
I run the same code on two websites. One website should contain Google AdSense block, when another shouldn't. Is there any way to disable Google AdSense on the second site, if showGoogleAd is false?
JavaScript:
var showGoogleAd = false;
HTML:
<script async src="//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js"></script>
<!-- vkkar.ru -->
<ins class="adsbygoogle"
style="display:inline-block;width:728px;height:90px"
data-ad-client="ca-pub-1234567890123456"
data-ad-slot="1234567890"></ins>
<script>
(adsbygoogle = window.adsbygoogle || []).push({});
</script>
I know that I can load script based on boolean value -
if (showGoogleAd) {
$.getScript('//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js', function(){});
But what should I do with remaining part? Or, can I just leave it?
Looks like it is enough to remove the following line from the html:
<script async src="//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js"></script>
and load it with JavaScript, if ad should be shown -
if (showGoogleAd) {
$.getScript("//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js");
}
i test it on my website to disable Google AdSense, i use below script that remove ads ?
*
<script async src="//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js"></script>
<!-- Resposive_TEST -->
<ins class="adsbygoogle responsive-test"
data-ad-client="ca-pub-3086914080036003"
data-ad-slot="1408862175"
data-ad-format="auto"></ins>
<script>
(adsbygoogle = window.adsbygoogle || []).push({});
</script><pre> <code>
*
Without Google Ads, my HTTPS webpage loads in about 500ms. With Google Ads, the same webpage takes 2-5 seconds to load. I have two banner ads (one at the top and one at the bottom). Yes, it renders the entire page before the ads, but I still think it's clunky to have the browser spinning while it waits for the ads to finish.
Is there any way with Javascript to finish page loading before the ads, and then just load the ads in the background? Basically, trick it into thinking the entire page is already loaded?
Current code:
<script async src="//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js"></script>
<!-- My Ad Banner -->
<ins class="adsbygoogle"
style="display:inline-block;width:728px;height:90px"
data-ad-client="myid"
data-ad-slot="myid"></ins>
<script>
(adsbygoogle = window.adsbygoogle || []).push({});
</script>
Looking at the waterfall for my site, it doesn't look like the adsbygoogle.js is causing the load indicator. Instead, it's the actual ad content itself. For example, this object (the banner ad) doesn't even start loading until 1.8 seconds (long after my entire page has already loaded): tpc.googlesyndication.com/simgad/AdID
Thanks!
Try this article.
The trick is to put your ad initializing and loading logic in window's onload event:
<script>
window.onload = function(){
...
};
</script>
This is my ultimate Vanilla JS solution:
<script async defer src="https://pagead2.googlesyndication.com/pagead/js/adsbygoogle.js"></script>
<script>
(adsbygoogle = window.adsbygoogle || []).onload = function () {
[].forEach.call(document.getElementsByClassName('adsbygoogle'), function () {
adsbygoogle.push({})
})
}
</script>
A Vanilla solution is to leave all your ad code as is, except to remove the script tag, and then add to your javascript
function loadGoogleAds() {
var scriptTag = document.createElement('script');
scriptTag.setAttribute('src', '//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js');
scriptTag.setAttribute('type', 'text/javascript');
scriptTag.setAttribute('async', 'async');
document.body.appendChild(scriptTag);
}
window.addEventListener("load", loadGoogleAds);
Disclaimer: as with the other answers to this question it is assumed that Google does not disapprove of methods to improve page performance
Ads_onscroll_event
Naturally, this is how an original ad unit code look.
<script async src="//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js"></script>
<!-- leaderboard -->
<ins class="adsbygoogle"
style="display:inline-block;width:728px;height:90px"
data-ad-client="ca-pub-xxxxxxxxxxxxxxxx"
data-ad-slot="1234567890"></ins>
<script>
(adsbygoogle = window.adsbygoogle || []).push({});
</script>
First of all, remove below script from all existing ad units.
<script async src="//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js"></script>
And
<script>
(adsbygoogle = window.adsbygoogle || []).push({});
</script>
Now it will appear like this
<ins class="adsbygoogle"
style="display:inline-block;width:728px;height:90px"
data-ad-client="ca-pub-xxxxxxxxxxxxxxxx"
data-ad-slot="1234567890"></ins>
This approach will load ad unit once user scroll your web page add this on page that has ads:
<script type="text/javascript">
//<![CDATA[
var la=!1;window.addEventListener("scroll",function(){(0!=document.documentElement.scrollTop&&!1===la||0!=document.body.scrollTop&&!1===la)&& (!function(){var e=document.createElement("script");e.type="text/javascript",e.async=!0,e.src="https://pagead2.googlesyndication.com/pagead/js/adsbygoogle.js";var a=document.getElementsByTagName("script")[0];a.parentNode.insertBefore(e,a)}(),la=!0)},!0);//]]>
</script>
And
<script>
(adsbygoogle = window.adsbygoogle || []).onload = function () {
[].forEach.call(document.getElementsByClassName('adsbygoogle'), function () {
adsbygoogle.push({})
})
}
</script>
No matter how many google ads you have on the page, put ONE copy of the external js within the head of your page ... it's OK at the top since it's async.
Put the ins supplied by google where the ad is to appear.
At the bottom of your page, just before the end body tag, put the script to push the ad.
Now the ad will load after your page is complete !
If you want to add a second adsense ad on the page, do not repeat the adsbygoogle.js (one copy is enough) ... put the second where it is to be displayed ... add another push at the bottom of your page so it looks like this:
(adsbygoogle = window.adsbygoogle || []).push({});
(adsbygoogle = window.adsbygoogle || []).push({});
Place the code for initiating the ad's at the bottom of the page just before the closing /body> tag
This topic has been already answered here under lazy load topic, But this is actually defer loading.
How to lazy load new Google Adsense code using JS
<script>
function downloadJSAtOnload() {
var element = document.createElement("script");
element.src = "https://pagead2.googlesyndication.com/pagead/js/adsbygoogle.js?client=ca-pub-XXXXXXXXXXXXXXXX";
element.async = true;
element.setAttribute('crossorigin', 'anonymous');
document.body.appendChild(element);
}
if (window.addEventListener)
window.addEventListener("load", downloadJSAtOnload, false);
else if (window.attachEvent)
window.attachEvent("onload", downloadJSAtOnload);
else window.onload = downloadJSAtOnload;
</script>
This JavaScript is cross-browser compatible and works with old browsers too.
If you are looking for Lazy Loading Adsense Ads, Check this answer
How to Lazy Load Adsense Ads?
(This solution requires jQuery)
Remove the <script> tag from your code and add this to your javascript:
$(function() {
$.getScript("//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js");
});
This will do an AJAX call to get the script, rather than using the async browser method. This should result in not having the loading indicator.
ref: http://api.jquery.com/jquery.getscript/
This is what I did, after noticing adsense slows down page loading/rendering:
first of all add this code to your <Body> tag:
<body onLoad="initialize_page();">
Now, put only one adsense script code, above the <HEAD> tag:
<script async src="//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js"></script>
Remove adsense <script async src="//pagead2.googlesyndication.com/pagead/js/adsbygoogle.js"></script> from all of your adsense code(one copy at the header is enough).
Remove all (adsbygoogle = window.adsbygoogle || []).push({}); from your adsense code from your page. We will be using them over our initialize function.
To our initialize_page() function, add this code:
function initialize_page()
{
(adsbygoogle = window.adsbygoogle || []).push({});//this is for the first adsense
(adsbygoogle = window.adsbygoogle || []).push({});//this is for the second
(adsbygoogle = window.adsbygoogle || []).push({});//this is for the third
}
This code is suitable for 3 adsense on your page. if you use only 2 or 1 adesnes, just remove (adsbygoogle = window.adsbygoogle || []).push({}); from the function respectively.
I put
<script async src="https://pagead2.googlesyndication.com/pagead/js/adsbygoogle.js"></script>
just before closing the body tag and removed it from every adsense ad.
Ads show fine and the page loads very fast.
After many tests I found that sometimes ads don't show in safari.