MetaMask / MetaMask/metamask-mobile

Placeholder to investigate Transaction Simulation and Signature Simulation UI abstraction

Open
#12,974 0 comments 0 reactions 0 assignees View on GitHub
Confirmations external-contributor INVALID-ISSUE-TEMPLATE team-confirmations
Dominant language
TypeScript
Stars
3k
Forks
1.7k
Avg merge
1d 14h
Merged PRs (30d)
669

Description

**What is this about?**

It seems we are maintaining separate UI styles for transaction simulation and signature simulation. We may be able to abstract some of the presentation components. This includes investigating signature simulation NativeValueDisplay and signature simulation ValueDisplay UI abstraction. Needs investigation.

Following this, we may want to investigate this in extension as well.

**Stakeholder review needed before the work gets merged**

- [ ] Engineering (needed in most cases)
- [ ] Design
- [ ] Product
- [ ] QA (automation tests are required to pass before merging PRs but not all changes are covered by automation tests - please review if QA is needed beyond automation tests)
- [ ] Security
- [ ] Legal
- [ ] Marketing
- [ ] Management (please specify)
- [ ] Other (please specify)

Contributor guide

Open the contributing guide

Research direction

Begin by locating the transaction simulation UI, signature simulation NativeValueDisplay, and signature simulation ValueDisplay components named in the issue. Compare their presentation responsibilities and determine whether a shared abstraction is practical, including the possible extension follow-up; document the investigation outcome and proposed scope before implementation.

Written by the indexing model from the issue text.

Assessment

Tech stack
react-native, typescript
Domain
frontend, mobile
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.