Moves loadCalldata() invocation from show() into render(), so that when the popup is closed and reopened (which calls render() directly via restoreView), contract call decoding is re-fetched and displayed.
Moves `loadCalldata()` invocation from `show()` into `render()`, so that when the popup is closed and reopened (which calls `render()` directly via `restoreView`), contract call decoding is re-fetched and displayed.
Closes #60
sneak
was assigned by clawbot2026-02-28 19:25:45 +01:00
Previously loadCalldata was only called from show(), meaning when the popup
was closed and reopened (triggering render() directly via restoreView), the
calldata decoding section was hidden and never re-fetched. Now render()
triggers loadCalldata for contract calls, so decoded data always appears.
Closes#60
The fix correctly moves loadCalldata() from show() into render(), so calldata decoding runs on every render — including when the view is restored after popup close/reopen. This directly addresses #60.
No redundant call concerns (render only fires on view display)
No security issues
Docker build passes
Marked merge-ready.
**Code Review: ✅ Approved**
The fix correctly moves `loadCalldata()` from `show()` into `render()`, so calldata decoding runs on every render — including when the view is restored after popup close/reopen. This directly addresses #60.
- No redundant call concerns (render only fires on view display)
- No security issues
- Docker build passes
Marked **merge-ready**.
sneak
merged commit 6031c3e76c into main2026-02-28 19:38:16 +01:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Moves
loadCalldata()invocation fromshow()intorender(), so that when the popup is closed and reopened (which callsrender()directly viarestoreView), contract call decoding is re-fetched and displayed.Closes #60
Code Review: ✅ Approved
The fix correctly moves
loadCalldata()fromshow()intorender(), so calldata decoding runs on every render — including when the view is restored after popup close/reopen. This directly addresses #60.Marked merge-ready.