@bx80 opened this Pull Request on October 25th 2021 Contributor

Description:

Fixes #15429
Added setup guide links for Wordpress, Squarespace, Wix, SharePoint, Joomla, Shopify and Google Tag Manager to the 'No data' email and to the tracking code page.

Review

@sgiehl commented on October 26th 2021 Member

@tsteur @Findus23 Not sure on that one, but do we still need to add rel="noreferrer noopener" to new links, or is that meanwhile covered by meta tags or headers?

@tsteur commented on October 27th 2021 Member

noreferrer we wouldn't need anymore AFAIK. noopener is used by default in FF 79 https://bugzilla.mozilla.org/show_bug.cgi?id=1522083 , safari 68 https://webkit.org/blog/8475/release-notes-for-safari-technology-preview-68/ brave: https://github.com/brave/brave-browser/issues/1840

Here's a detailed list of supported browsers: https://caniuse.com/mdn-html_elements_area_implicit_noopener

Seems maybe opera and android browser doesn't have this fixed yet and maybe still need it for a while? Also samsung browser only seems to have it since recently etc.

This Pull Request was closed on October 28th 2021
Powered by GitHub Issue Mirror