Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue with ad clicking behaviour: target changed from _blank to _top #40193

Closed
suvi-salo opened this issue Oct 25, 2024 · 4 comments
Closed

Issue with ad clicking behaviour: target changed from _blank to _top #40193

suvi-salo opened this issue Oct 25, 2024 · 4 comments

Comments

@suvi-salo
Copy link

Hi,

We have a wide, several GAM networks affected, issue where if you click on an ad on desktop, it opens in the same window instead of a new one. On GAM, all the settings point to _blank but amp-ad-exit mjs changes it to _top. We were in touch with GAM Publisher support and they advised us to contact you here. One of the affected sites is byggebolig.no. Let me know if you need any further information from us.

@erwinmombay erwinmombay self-assigned this Oct 30, 2024
@erwinmombay
Copy link
Member

heya @suvi-salo . do you happen to have an example to demonstrate the behavior?

@suvi-salo
Copy link
Author

@erwinmombay If you take a look at byggebolig.no and click on any ad, you'll see it opens in the same window rather than in a new tab.
image
image

This is what we see in amp-ad-exit-0.1.mjs file, which causes the behaviour:
image

@erwinmombay erwinmombay assigned powerivq and unassigned erwinmombay Oct 31, 2024
@powerivq
Copy link
Contributor

powerivq commented Nov 1, 2024

@suvi-salo Looking at your amp-ad-exit configuration, it specified that the link is to be opened with _top (see screenshot below) , and that overrides the a tag, so everything is working as intended from AMP's perspective.

image

If you have questions about the amp-ad-exit configuration, you would need to contact GAM, as AMP only renders the creative as instructed.

@erwinmombay
Copy link
Member

closing out for now. feel free to re-open if there is any further discussion needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants