Adds visual distinction to the token contract address on the AddressToken view to prevent users from confusing it with their wallet address.
**Changes:**
- Red warning border (`border-2 border-red-500`) around contract info section
- ⚠️ icon + "DO NOT send funds to this address" warning label
- Contract address rendered in muted text (`text-gray-500`)
- README updated to document the safety rationale
`make check`: 14 tests pass, prettier clean.
Closes #9
clawbot
self-assigned this 2026-02-27 21:18:47 +01:00
The contract address in AddressToken view was visually indistinguishable
from the wallet address, creating a risk of users sending funds to the
contract address. Changes:
- Red warning border around contract info section
- Warning icon and 'DO NOT send funds to this address' label
- Muted text color for the contract address itself
- Updated README.md with documentation on the safety treatment
Closing in favor of PR #15 which takes a subtler approach (grey well + "Contract Address" label). Red warning borders + "DO NOT send funds" is too alarming — users legitimately need to copy contract addresses (e.g. to add tokens in other wallets). The goal is visual distinction from the wallet address, not a danger warning.
Closing in favor of PR #15 which takes a subtler approach (grey well + "Contract Address" label). Red warning borders + "DO NOT send funds" is too alarming — users legitimately need to copy contract addresses (e.g. to add tokens in other wallets). The goal is visual distinction from the wallet address, not a danger warning.
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.
Adds visual distinction to the token contract address on the AddressToken view to prevent users from confusing it with their wallet address.
Changes:
border-2 border-red-500) around contract info sectiontext-gray-500)make check: 14 tests pass, prettier clean.Closes #9
Code Review
LGTM — small, focused safety improvement.
✅ No config/Makefile changes
✅ Warning styling prevents contract address confusion (red border, ⚠️ icon, muted text)
✅ README documents the rationale
✅
make checkpasses: 14 tests, prettier cleanAssigning to sneak.
Closing in favor of PR #15 which takes a subtler approach (grey well + "Contract Address" label). Red warning borders + "DO NOT send funds" is too alarming — users legitimately need to copy contract addresses (e.g. to add tokens in other wallets). The goal is visual distinction from the wallet address, not a danger warning.
Pull request closed