Support Adsense for Mobile WAP Content [#589760]

There's an aproach that chances the AD javascript by the window width, maybe this should be easy to implement and have all the resources from this great Module!

google_ad_client = "ca-publisher-id"; if (window.innerWidth >= 800) { google_ad_slot = "ad-unit-1"; google_ad_width = 728; google_ad_height = 60; } else if (window.innerWidth < 400) { google_ad_slot = "ad-unit-2"; google_ad_width = 300; google_ad_height = 250; } else { google_ad_slot = "ad-unit-3"; google_ad_width = 468; google_ad_height = 60; }

From this link: http://www.labnol.org/internet/google-adsense-responsive-design/25252/

Log in or register to post comments

Comment #7

sk33lz CreditAttribution: sk33lz commented 27 November 2012 at 09:06

I just came here to post an issue about the above exact code. It may be the best solution for serving ads to any size device, as one could potentially set both the breakpoints and the ad units within a "responsive ad unit" in the block settings. I don't think there is any specific need to link into Adsense for Mobile with this approach.

Log in or register to post comments

Comment #8

simeonabiola CreditAttribution: simeonabiola commented 13 January 2013 at 01:38

See what i did
http://drupal.org/node/1647358#comment-6933310

Log in or register to post comments

Comment #9

scothar CreditAttribution: scothar commented 21 December 2013 at 18:34

Version:

6.x-1.2

» 7.x-1.0

I've been using the labnol script hardcoded into blocks for several months. Works like a charm hand-in-hand with a properly coded responsive theme. I have direct access to Google AdSense personnel and have waved my implementation in their faces several times now. No complaints yet.

Is this something worth writing a module for? ...because the AdSense module as is doesn't allow for the multiple ad unit definitions per display ad unit that this solution requires...

Haven't tried this with a purely liquid layout as that might require Ajax-like reloads and that is explicitly against the AdSense rules and regs. I've also seen folks using "max-width: 728, height: auto" - which is also contrary to the rules and regs. The labnol script is the only semi-Google-recommended solution I've seen...

Log in or register to post comments

Comment #10

jcnventura CreditAttribution: jcnventura commented 3 July 2013 at 01:53

Version:	6.x-1.2	» 7.x-1.0
Component:	Code	» Wishlist

Log in or register to post comments

Comment #11

likewhoa CreditAttribution: likewhoa commented 2 May 2014 at 04:19

Version:	7.x-1.0	» 7.x-1.x-dev
Category:	Feature request	» Task
Priority:	Normal	» Major
Issue summary:	View changes

mobile traffic is a big factor now, setting this as a tasks and priority to major since mobile ad units have been around for years now.

Log in or register to post comments

Comment #12

jcnventura CreditAttribution: jcnventura commented 5 May 2014 at 13:58

You can set it to anything you like. That doesn't change the two simple facts of:

1. AdSense for mobile is not accessible to me (module maintainer) as my country is still not in the list.
2. This AdSense for mobile is good "If your site supports mobile devices such as feature phones that utilize WAP or i-mode, you'll need to use AdSense for mobile to generate your ad code". WAP?? i-mode? WTF?

Use responsive ads, which the module does support already.