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

token contract display on 'confirm-tx' page is inconsistent with token contract display elsewhere #70

New Issue
Closed
opened 2026-02-28 20:56:45 +01:00 by sneak · 1 comment
sneak commented 2026-02-28 20:56:45 +01:00
Owner
Copy Link

we strive for UI data consistency in all things. please identify any other places where the same types of data (such as contract addresses - we use a well) are displayed inconsistently on different views.

we strive for UI data consistency in all things. please identify any other places where the same types of data (such as contract addresses - we use a well) are displayed inconsistently on different views.
sneak added the
bot
 label 2026-02-28 20:56:50 +01:00
clawbot self-assigned this 2026-02-28 21:03:23 +01:00
clawbot removed their assignment 2026-02-28 21:05:45 +01:00
clawbot commented 2026-02-28 21:06:53 +01:00
Collaborator
Copy Link

Inconsistency Audit & Fix

Primary issue (confirm-tx)

The confirm-tx page displayed token contract addresses as plain text with just an etherscan link, while addressToken used a styled bg-hover rounded-md container with:

  • Color dot (addressDotHtml)
  • Copy-on-click (dashed underline)
  • Etherscan external link
  • Token metadata (name, symbol)

Fixed: confirm-tx now uses the same styled container pattern with all of the above.

Full view audit

View Contract display Status
addressToken Styled container with dot, copy, link, metadata ✅ Reference
confirmTx Was plain text, now styled container ✅ Fixed
approval Simpler display, appropriate for dapp context ✅ OK
txStatus Address display consistent with context ✅ OK
transactionDetail Uses well pattern for calldata ✅ OK
send Input form, no contract display ✅ N/A

No additional inconsistencies found that warrant changes.

PR: #74

## Inconsistency Audit & Fix ### Primary issue (confirm-tx) The confirm-tx page displayed token contract addresses as plain text with just an etherscan link, while addressToken used a styled `bg-hover rounded-md` container with: - Color dot (`addressDotHtml`) - Copy-on-click (dashed underline) - Etherscan external link - Token metadata (name, symbol) **Fixed**: confirm-tx now uses the same styled container pattern with all of the above. ### Full view audit | View | Contract display | Status | |------|-----------------|--------| | addressToken | Styled container with dot, copy, link, metadata | ✅ Reference | | confirmTx | Was plain text, now styled container | ✅ Fixed | | approval | Simpler display, appropriate for dapp context | ✅ OK | | txStatus | Address display consistent with context | ✅ OK | | transactionDetail | Uses well pattern for calldata | ✅ OK | | send | Input form, no contract display | ✅ N/A | No additional inconsistencies found that warrant changes. PR: #74
clawbot self-assigned this 2026-02-28 21:12:20 +01:00
clawbot removed their assignment 2026-02-28 21:12:32 +01:00
clawbot removed the
bot
 label 2026-02-28 21:31:59 +01:00
sneak closed this issue 2026-02-28 21:33:54 +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
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
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#70
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?