Google recently launched a new version of the AdSense code. The new AdSense code allows AdSense to trigger optimization features earlier which can improve the performance of the ads on your site. The new ad code works for both Auto ads and ad units.
Auto ads code example:
<script async src="https://pagead2.googlesyndication.com/pagead/js/adsbygoogle.js?client=ca-pub-1234" crossorigin="anonymous"</script>
Ad unit code example:
<script async src="https://pagead2.googlesyndication.com/pagead/js/adsbygoogle.js?client=ca-pub-1234" crossorigin="anonymous"</script>
<ins class="adsbygoogle"
style="display:block"
data-ad-client="ca-pub-1234"
data-ad-slot="5678"
data-ad-format="auto"
data-full-width-responsive="true"></ins>
<script>
(adsbygoogle = window.adsbygoogle || []).push({});
</script>| Comment | File | Size | Author |
|---|---|---|---|
| #7 | lighthouse_report_adsense_async_with_client_parameter.pdf | 243.81 KB | it-cru |
| #7 | lighthouse_report_adsense_defer_with_client_parameter.pdf | 253.13 KB | it-cru |
Show commandsStart within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes, plain diff MR !1
Comments
Comment #3
jcnventuraComment #4
it-cru@jcnventura: Would it not be better to use hook_library_info_alter() to add dynamic configured client=ca-pub-1234 URL parameter to adsense.adsbygoogle JS library, use library than via TWIGs attach_library('adsense/adsense.adsbygoogle') and get rid of defer variant?
I currently testing hard-coded client=ca-pub-1234 URL parameter with async on one of my sites and it brings better results in Google's Lighthouse reporting than my previous provided defer solution.
What do you think?
Comment #5
jcnventuraHard to understand what you mean here. So the new code is better than with defer, and that option should be removed?
Comment #6
it-cru@jcnventura: What I currently did is to add my ca-pub-1234 as client URL parameter manually to adsense.adsbygoogle library for first testings and switched from defer back to async, but with usage of TWIGs attach_library() calls. So I got new Adsense script into head.
As far as I understand we don't need this script added to each Adsense ad anymore. See also https://support.google.com/adsense/answer/9274516?hl=en
I haven't tested new Adsense script with client URL parameter together with defer option. I will try to test this and if I have time I will also try to provide a patch with usage of hook_library_info_alter() to add client URL parameter dynamically to the adsense.adsbygoogle library. Maybe also with an optional switch from async to defer.
Comment #7
it-cru@jcnventura: I've opened a MR related to this issue https://git.drupalcode.org/project/adsense/-/merge_requests/1 which use hook_library_info_alter() to add client URL parameter and set defer if it is enabled.
I've attached 2 Google Lighthouse reports which currently show no big difference between usage of async or defer in Adsense JS included in head on my site. So no clue if defer option is really required anymore or not.