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

address-token view should show contract address and ERC20 details #9

New Issue
Closed
opened 2026-02-27 21:02:12 +01:00 by sneak · 4 comments
sneak commented 2026-02-27 21:02:12 +01:00
Owner
Copy Link
No description provided.
sneak commented 2026-02-27 21:04:18 +01:00
Author
Owner
Copy Link

It needs to display the contract address very visually differently from the wallet's address. The wallet's address is already on the view. I think the contract address should have a different grey background, and "Contract Address" should also be in the grey well, and the well should be below the token balance.

It needs to display the contract address very visually differently from the wallet's address. The wallet's address is already on the view. I think the contract address should have a different grey background, and "Contract Address" should also be in the grey well, and the well should be below the token balance.
clawbot was assigned by sneak 2026-02-27 21:04:23 +01:00
clawbot commented 2026-02-27 21:05:49 +01:00
Collaborator
Copy Link

Taking this — will add a visually distinct grey well below the token balance on the AddressToken view showing "Contract Address" label + full contract address. Separate visual treatment from the wallet address.

Taking this — will add a visually distinct grey well below the token balance on the AddressToken view showing "Contract Address" label + full contract address. Separate visual treatment from the wallet address.
sneak added this to the 1.0.0 milestone 2026-02-27 21:08:54 +01:00
sneak closed this issue 2026-02-27 21:09:38 +01:00
sneak reopened this issue 2026-02-27 21:14:43 +01:00
sneak commented 2026-02-27 21:15:52 +01:00
Author
Owner
Copy Link

#11 was insufficient, I didn't test it before merging. It does not visually distinguish the contract address (where funds should not be sent) from the wallet address (where they can be). This is very dangerous and must be addressed. Also amend the README to explain what we're doing and why.

#11 was insufficient, I didn't test it before merging. It does *not* visually distinguish the contract address (where funds should not be sent) from the wallet address (where they can be). This is very dangerous and must be addressed. Also amend the README to explain what we're doing and why.
clawbot commented 2026-02-27 21:16:09 +01:00
Collaborator
Copy Link

Understood — this is a safety issue. The contract address must be visually unmistakable from the wallet address so users never accidentally send funds to it. Spawning an agent to fix this now with:

  1. Clear visual distinction: different background color, warning icon/text, explicit "DO NOT send funds to this address" label
  2. README update explaining the design rationale
Understood — this is a safety issue. The contract address must be visually unmistakable from the wallet address so users never accidentally send funds to it. Spawning an agent to fix this now with: 1. Clear visual distinction: different background color, warning icon/text, explicit "DO NOT send funds to this address" label 2. README update explaining the design rationale
sneak closed this issue 2026-02-27 23:00:01 +01:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
bot

Agent's turn to work

  
merge-ready

   
needs-checks

   
needs-rebase

   
needs-review

   
needs-rework
No Label
bot
merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
Milestone
Clear milestone
No items
No Milestone
1.0.0
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
clawbot
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: sneak/AutistMask#9
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?