MetaMask / MetaMask/metamask-mobile
[Bug]: Confirmations - Advanced Details get's cut (no close button) if I open the individual transaction details from a batch
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 3k
- Forks
- 1.7k
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 669
Description
Describe the bug
Advanced Details get's cut (no close button) if I open the individual transaction details from a batch.
That view is not scrollable, so if I want the Close button to appear again, I need to fold again the transaction items
Screenshots/Recordings
https://github.com/user-attachments/assets/69f586f3-84b5-4d40-a12b-1f53216b3f5e
Steps to reproduce
- Go to test dapp
- Trigger a batch tx with multiple tx (you must have a Smart Account enabled for that network)
- Click Advanced details
- Unfold each transaction
- See the Close button is not in the screen anymore and you cannot scroll on that view
Error messages or log output
Where was this bug found?
Live version (from official store)
Version
7.70.0
Build number
4130
Build type
None
Device
Pixel 6
Operating system
Android
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 by reproducing the issue in the test dapp on Android with a Smart Account enabled, then open Advanced details for a batch transaction and unfold every transaction. Inspect the individual transaction details view and its layout behavior. Done means the Close button remains accessible, or the view can scroll to it, after all transaction items are unfolded.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react-native, typescript
- Domain
- frontend, mobile
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100