sneak/AutistMask
1
0
Fork 0
Code Issues 67 Pull Requests 3 Actions Packages Projects Releases Wiki Activity

feat: show red warning when sending to address with zero tx history #98

Merged
sneak merged 6 commits from issue-82-zero-tx-warning into main 2026-03-01 00:54:15 +01:00
Conversation 12 Commits 6 Files Changed 2
+37
6 Commits
Author SHA1 Message Date
sneak 1fb9fade51 Merge branch 'main' into issue-82-zero-tx-warning
check / check (push) Successful in 22s
Details
2026-03-01 00:53:45 +01:00
user 045328f3b9 fix: use visibility:hidden/visible instead of CSS transitions for zero-tx warning
check / check (push) Successful in 22s
Details 
Remove all CSS transitions, max-height changes, and opacity animations.
The warning container always reserves its space with visibility:hidden
and switches to visibility:visible when needed. No layout shift ever.
 2026-02-28 15:46:58 -08:00
user 576fe3ab15 fix: replace visibility:hidden with smooth collapse for zero-tx warning
check / check (push) Successful in 10s
Details 
Instead of permanently reserving space with visibility:hidden, the warning
container now uses max-height + opacity transitions. Space is reserved during
the async check, then smoothly collapses to 0 if the warning isn't needed.
This reclaims ~40px of popup viewport in the common case.
 2026-02-28 15:37:27 -08:00
user 8c071ae508 fix: never collapse warning container — always reserve space to prevent layout shift
check / check (push) Successful in 10s
Details 
Replace display:none with persistent visibility:hidden so the warning
area occupies the same vertical space regardless of API result.
This eliminates the layout shift that occurred when the container was
collapsed after the recipient history check returned.
 2026-02-28 15:26:49 -08:00
user a3c2b8227a fix: zero-tx warning layout shift and contract address false positive 
- Reserve space for the warning upfront using visibility:hidden instead
  of display:none, preventing layout shift per README policy
- Move warning HTML to index.html as a static element rather than
  injecting dynamically
- Skip warning for contract addresses (check getCode first) since
  getTransactionCount only returns outgoing tx nonce
- Collapse reserved space when warning is not needed (address has
  history, is a contract, or on RPC error)
 2026-02-28 15:26:44 -08:00
user f9f3e7b85a feat: show red warning when sending to address with zero tx history 
On the confirm-tx view, asynchronously check the recipient address
transaction count via getTransactionCount(). If zero, display a
prominent red warning advising the user to double-check the address.

Closes #82
 2026-02-28 15:26:44 -08:00