after them upgrade skin (wallpaper ad) is no longer click-able

Posted in: Newsmag
Post count: 47

Hi
after the recent theme upgrade our wallpaper ad is no longer clickable.

We receive display ads through a our ad server. To allow the support of the wallpaper (skin) ad, I’ve followed a your suggestion and I’ve added a code change like the one described here:
https://forum.tagdiv.com/topic/wallpaper-ad-to-be-fed-via-ad-server/

Unfortunately now the image appears correctly, but it cannot be clicked, probably because of a layer in front of it.

This is the upgraded (production) site: http://www.calcioefinanza.it
this is the previous (test but working) version: test.calcioefinanza.it

Can you please check?

Best regards
Fabio

Post count: 6535

Hi

I assume that there is a html issue. Make sure that the html tags are properly closed and also check if the changes are made as I’ve indicated in my previous topic. That solution still work with V 3.2 of Newsmag theme and if it’s properly implemented there shouldn’t be any issues with it.

Thanks!

Post count: 47

Thanks,
what I’ve done is to add these lines of code on the tail of header.php

?>

<div class="left">
<?php echo do_shortcode ('[cfAdZoneCode zoneid="21" zonename="0.1 skin"]') /* FV Change skin ad */ ?>
</div>

I’ve seen some changes in the new version, maybe this must be changed?

Post count: 6535

HI

Try to insert the ad code in one of theme’s ad spots then add inside do_shortcode function the shortcode of that ad-spot.
Note that you may also need additional customization to make the ads work there. We can’t offer customization services for the moment, most we can do is to point a possible way to achieve that. As I’ve mentioned in my previews reply the code works fine on my end with a banner ad, I mean that the ad is clickable and works fine.

Thanks!

Post count: 47

Hi Andrei,
can you just check why the skin is no longer clickable?
You can check the 2 versions here, as written:
After theme upgrade (production, skin not clickable): http://www.calcioefinanza.it
Before theme upgrade (test site, skin clickable): http://test.calcioefinanza.it
Probably it’s just matter to fix the size of a div that is on top of the skin div, preventing it to be clicked.

In this case would be a css issue, easy to fix…

Post count: 47

By the way, I’ve found that if I set this CSS property of id=”td-outer-wrap” style=”pointer-events: none;”

it seems to work… do you see any possible side effect?

Post count: 6535

Hi

Yes it may affect the lower containers: but you can’t use this css to prevent that:

.td-container{
pointer-events: auto;
}

Thanks!

Post count: 47

I suppose you meant “you CAN use this css…” . And it worked!

Thanks
Regards

Post count: 47

I have to rollback my comment.
Adding this CSS rules

#td-outer-wrap {
pointer-events: none;
}

.td-container{
pointer-events: auto;
}

The skin is clickable, the .td-container classified objects are clickable, but other are passed through

For instance the site logo and the top menus are not clickable and the effect to click them is to click the SKIN

Do you have other suggestions?

Post count: 6535

Hi

yes indeed I meant that you can use the above css, sorry for that mistake.
In addition you need to include the header container so try this code, it should work fine:

#td-outer-wrap {
pointer-events: none;
}
.td-header-container,
.td-container{
pointer-events: auto;
}

Thanks!

Viewing 10 posts - 1 through 10 (of 10 total)
The forum ‘Newsmag’ is closed to new topics and replies.