MetaMask / MetaMask/metamask-extension
Remove use of unnecessary target and rel attributes where possible
Open
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
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