Hi,
I have a little enhancement to assist ad placement during development. I found it helpful for the placeholder text to include the block title and the dimensions, particularly when you have several blocks all designed for specific places on your site, using different visability settings. Having the title and dimensions makes it easier to see exactly which blocks are being shown in which locations.

There is one change in adsense_managed_block to add the title to the returned arry, and there is one change in adsense_display which adds the title (if available) and adds the dimensions if these are not already included in the title or description.

Patch coming up.

Comments

jonathan1055’s picture

Status: Active » Needs review
StatusFileSize
new121.41 KB
new98.38 KB
new2.04 KB

Here is the patch, against 1.4, and some screen shots.

Before

After

jcnventura’s picture

Status: Needs review » Fixed

Thanks for the patch. Added it to git.

http://drupalcode.org/project/adsense.git/commit/8415259

jonathan1055’s picture

THank you.

Status: Fixed » Closed (fixed)

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