Closed (fixed)
Project:
Google AdSense integration
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
18 Nov 2012 at 13:42 UTC
Updated:
14 Jul 2013 at 01:41 UTC
Jump to comment: Most recent file
Comments
Comment #1
jursx commentedYes please add ...
Comment #2
philsward commentedI looked into the issue again and figured out how to add the 300x600 Large Skyscraper to the module code. I must have missed a step last time I looked into it...
1) Create your 300x600 ad in adsense
2) go to "/sites/all/modules/adsense/" and edit adsense.module
3) Look around line 51 for:
4) go to the very bottom of the ad format setup. Copy the very last ad format:
then paste it underneath itself, so you have a duplicate.
5) Next change everything in the second copy to reflect the 300 x 600 ad format (or just copy and paste the following):
6) Save it, that part is done
7) Add the adsense code to your page: [adsense:300x600:{your ad code#}]
8) Flush your cache
9) Check on browser that isn't logged in. If you did it right, the first few times you check the page it will probably be blank. as long as you aren't seeing the code identifier [adsense:300x600....] you should be all set.
I don't have a clue how to create a patch for this... so anyone else who want's to give it a stab, feel free.
Comment #3
fenstratHere it is in patch format as per what was done in #1603174: Add 320 x 50 Mobile Banner size.
Comment #4
jcnventuraSorry for the time it took. I've now committed it to git.
Comment #5
fenstratNo worries @jcnventura, thanks for committing.