MetaMask / MetaMask/metamask-extension
UI - Confirm Tx - Show parsed data when ABI is available
- Dominant language
- TypeScript
- Stars
- 13.2k
- Forks
- 5.6k
- Avg merge
- 2d 5h
- Merged PRs (30d)
- 451
Description
On the Tx confirm screen in the `data` section we show (1) the relevant ABI and (2) the raw data
If we have the ABI then we should just show the raw data parsed with the ABI
e.g.
```
arg0: 0xabcd...
arg1: 0xabcd...
arg2: 42
```
We can specify the parameter names in ABIs but they will not always be there so we need to have fallbacks (e.g. `arg0`).
For addresses, could use fancy address display with identicon + copy tooltips
Contributor guide
Research direction
Start by locating the Tx confirm screen and its data section, then inspect how the ABI and raw transaction data are currently presented. Check examples with ABI parameter names and without them, including address values. Done means ABI-backed parsed data replaces the separate ABI and raw-data display, with fallback argument labels and usable address presentation.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- blockchain, frontend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100