Ads do not display while using Internet Explorer. This is true for all user types (administrator...authenticated user...anonymous user).

Comments

jboese’s picture

it looks like for IE it is required for each block to provide the width and height for the advertisement, even though the module does not require these fields.

I originally chose to left this blank, as I don't like stretching ads smaller than this specification to fit the space as the SimpleAds module does...

jboese’s picture

Based on how this module assigns img width and height values, and how IE interprets them, i propose a patch to simpleads_img_element.tpl.php that does the following:

-<img src="<?php print $ad['image']; ?>" alt="<?php print check_plain($ad['alt']); ?>" width="<?php print check_plain($settings['ads_width']); ?>" height="<?php print check_plain($settings['ads_height']); ?>" border="0">

+<img src="<?php print $ad['image']; ?>" alt="<?php print check_plain($ad['alt']); ?>" <?php if ($settings['ads_width']){?>width="<?php print check_plain($settings['ads_width']); ?>"<?php } ?> <?php if ($settings['ads_height']) { ?>height="<?php print check_plain($settings['ads_height']); ?>"<?php } ?> border="0">

(Sorry about this. I haven't activated Git access yet. Doing that now :))

minnur’s picture

@jboese, thanks for finding this issue. I will fix it as soon as it possible. Most likely this weekend.
For the time being you may copy template file to your theme and modify it the way you want.

minnur’s picture

Assigned: Unassigned » minnur
Status: Active » Needs work
minnur’s picture

Priority: Major » Normal
Status: Needs work » Needs review
minnur’s picture

Status: Needs review » Needs work
minnur’s picture

Status: Needs work » Fixed

@jboese, This is fixed. Please download version 7.x-1.6 and run update.php

minnur’s picture

Status: Fixed » Closed (fixed)
jboese’s picture

Thanks so much for your work on this module!