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

Request to support window.open() #93

Open
ycmou opened this issue Oct 4, 2021 · 2 comments
Open

Request to support window.open() #93

ycmou opened this issue Oct 4, 2021 · 2 comments

Comments

@ycmou
Copy link

ycmou commented Oct 4, 2021

Hello!

We would like to request PCM to support "window.open()" for decorating links with PCM attributes.

For context, because our website's click throughs needs to handle various other features like spam detection and event logging, we prevent the anchor tag's default behavior and instead ultimately delegate to use "window.open()" to navigate the user offsite. As such, we are not able to use anchor tag attributes to add decoration for PCM.

It would be great to have this support for us to fully integrate with PCM. For reference, Google's Attribution Reporting API offers it in this way: https://github.com/WICG/conversion-measurement-api/blob/main/event_attribution_reporting_clicks.md#registering-attribution-sources-for-windowopen-navigations, which does work for our use case.

One aspect to consider that was brought up by @johnwilander is to require "opt in to no opener so that the two windows cannot communicate after the second has been opened". See original thread in https://bugs.webkit.org/show_bug.cgi?id=231094.

Thanks!

@ycmou
Copy link
Author

ycmou commented Dec 1, 2021

Will this feature be supported in a future Safari update?

@johnwilander
Copy link
Collaborator

Hi! Thanks for filing. We avoid discussing individual browser plans here. But I’m tentatively in favor of this addition to the spec.

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

No branches or pull requests

3 participants
@johnwilander @ycmou and others