Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

Change skipTargetBlank to skipNoOpener #2

Closed
davidtheclark opened this issue Jun 30, 2017 · 2 comments
Closed

Change skipTargetBlank to skipNoOpener #2

davidtheclark opened this issue Jun 30, 2017 · 2 comments
Assignees

Comments

@davidtheclark
Copy link
Contributor

As @yoshuawuyts pointed out, rel="noopener" is preferred to target="_blank" for opening links in new tabs.

The option should apply to both patterns; but we should change the option name to favor the better pattern.

@davidtheclark davidtheclark self-assigned this Jun 30, 2017
@davidtheclark
Copy link
Contributor Author

I misunderstood the issue. rel="noopener" is supposed to be used in combination with target="_blank". On its own, it will not tell the browser to open a new tab.

https://mathiasbynens.github.io/rel-noopener/
https://developers.google.com/web/tools/lighthouse/audits/noopener

@yoshuawuyts
Copy link

yoshuawuyts commented Jun 30, 2017 via email

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

No branches or pull requests

2 participants