MetaMask / MetaMask/metamask-extension
Update to swaps price impact warning UX
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 13.2k
- Forks
- 5.6k
- Avg merge
- 2d 5h
- Merged PRs (30d)
- 451
Description
In order to reduce human error and ensure users actively acknowledge the possible price impact before executing a trade, we need to implement these designs: https://www.figma.com/file/CArPBnRG3IdQEv2Qji9Roo?embed_host=notion&kind=&node-id=3%3A506&viewer=1&viewport=991%2C139%2C0.25, https://www.figma.com/file/CArPBnRG3IdQEv2Qji9Roo?embed_host=notion&kind=&node-id=6%3A996&viewer=1&viewport=991%2C138%2C0.25
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the two linked Figma designs and locate the existing swaps price impact warning flow in the extension. Done means the warning UX matches the designs and users actively acknowledge the possible price impact before executing a trade.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- design, frontend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100