MetaMask / MetaMask/metamask-extension

Remove use of unnecessary target and rel attributes where possible

Open
#12,923 0 comments 0 reactions 0 assignees View on GitHub
needs-research
Dominant language
TypeScript
Stars
13.2k
Forks
5.6k
Avg merge
2d 5h
Merged PRs (30d)
451

Description

In a number of places we set the following on button and link components:
```
target="_blank"
rel="noopener noreferrer"
```

When we are opening a new window or tab with an `onClick` method, these are likely not needed. We should investigate and refactor this wherever possible in the codebase.

Contributor guide

Open the contributing guide

Research direction

Search the codebase for target="_blank" and rel="noopener noreferrer" on button and link components, then inspect the associated onClick handlers. Remove the attributes only where the handler opens a new window or tab without needing them, and verify that all necessary cases retain their existing behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
frontend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.