Change `allowReferrer` to `requireNoopener`

[benmccann]

Before You File a Bug Report Please Confirm You Have Done The Following...

• I have tried restarting my IDE and the issue persists.
• I have updated to the latest version of the packages.

What version of ESLint are you using?

any

What version of eslint-plugin-svelte are you using?

any

What did you do?

melt-ui/melt-ui#443 (comment)

rel="noreferrer"

What did you expect to happen?

rel="noreferrer" should not warn unless using svelte's legacy compiler option. We need a way to configure that

sveltejs/svelte#8230

What actually happened?

warned

Link to GitHub Repo with Minimal Reproducible Example

simply add rel="noreferrer". if you need a reproduction I can provide

Additional comments

No response

[ota-meshi]

It looks to me like you're using the svelte/no-target-blank rule yourself. Turn off rules when not needed 😉

https://sveltejs.github.io/eslint-plugin-svelte/rules/no-target-blank/

[benmccann]

oh, you are right! i'm sorry for this report. thank you so much for the help

[benmccann]

Thanks for the earlier advice. We tried to configure https://sveltejs.github.io/eslint-plugin-svelte/rules/no-target-blank/ appropriately, but were unable to do so (melt-ui/melt-ui#443 (comment)). I think the allowReferrer option isn't exactly correct because it allows you to write rel="noopener" when I think what you need to write is rel="noreferrer". We should probably replace the allowReferrer option with a requireNoopener option

[ota-meshi]

For clarity, could you please provide code examples of how the new option works and the correct and incorrect code when using the option?

By the way, the rule option is based on rule implemented for JSX.

https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-no-target-blank.md

[github-actions]

This issue is is stale because it missing information and has been open for 60 days with no activity.

[mcmxcdev]

Still relevant

[github-actions]

This issue is is stale because it missing information and has been open for 60 days with no activity.

[mcmxcdev]

Still relevant

[JounQin]

@mcmxcdev Please help to answer @ota-meshi's question instead of posting meaningless comments?