Unless I'm missing it, I'd like to see another token added for "Responsive Links".

I see [adsense:responsive:{slot}] which "only" covers text and[or] display by setting data-ad-format="auto", but nothing for the links.

Looking at the generated code, there is a switch to denote whether it is text and[or] display (auto) or link (link). I'm thinking this would have to either be another token created to specifically set it or an array(?) added to the existing token to denote that particular ad needs to be "link" instead of "auto". Skimming through the help docs and issue queue came up with nothing on the matter. From a UX standpoint for site builders, simply adding another token would be the most straight forward approach.

[adsense:responsive:{slot}]
[adsense:responsive-link:{slot}]

Examples:
Text and[or] Display

<!-- Responsive -->
<ins class="adsbygoogle"
     style="display:block"
     data-ad-client="ca-pub-xxxxxxxxxxxxxxxx"
     data-ad-slot="5431320244"
     data-ad-format="auto"></ins>
<script>
(adsbygoogle = window.adsbygoogle || []).push({});
</script>

Link

<!-- Responsive Links -->
<ins class="adsbygoogle"
     style="display:block"
     data-ad-client="ca-pub-xxxxxxxxxxxxxxxx"
     data-ad-slot="2632714949"
     data-ad-format="link"></ins>
<script>
(adsbygoogle = window.adsbygoogle || []).push({});
</script>

Comments

philsward created an issue. See original summary.

philsward’s picture

Issue summary: View changes
philsward’s picture

Issue summary: View changes

  • jcnventura committed fee8a6f on 8.x-1.x
    Issues #2810479 and #2838951: support matched content and responsive...

  • jcnventura committed a3c4f24 on 7.x-1.x
    Issues #2810479 and #2838951: support matched content and responsive...
jcnventura’s picture

Status: Active » Fixed
philsward’s picture

Sweet, thx @jcnventura, I'll check it out :)

Update: Looks like it's working perfectly!

  • jcnventura committed 77531e0 on 7.x-1.x
    Issues #2810479 and #2838951: better responsive ad detection
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.