fix: consistent contract address display across views (closes #70) #74

clawbot · 2026-02-28T21:06:42+01:00

clawbot commented

2026-02-28 21:06:42 +01:00

Summary

The confirm-tx page displayed token contract addresses as plain text while other views (addressToken) used a styled container with color dot, copy-on-click, etherscan link, and token metadata.

Changes

index.html: Updated confirm-token-section to use bg-hover rounded-md container with bold header, matching addressToken style
confirmTx.js: Render contract address with addressDotHtml color dot, dashed-underline copy-on-click span, etherscan link, and optional name/symbol metadata. Added click-to-copy handler.
send.js: Pass tokenName to confirmTx for richer contract info display. Import TOKEN_BY_ADDRESS for name resolution.

Audit Results

Audited all views for display inconsistencies:

confirm-tx: Token contract was plain text — now uses styled container ✅
addressToken: Already uses proper styled container (reference implementation)
approval.js: Uses simpler address display but appropriate for dapp approval context
txStatus.js: Uses address display consistent with its context
transactionDetail.js: Uses well pattern for calldata display, consistent

No other contract address display inconsistencies found that warrant changes in this PR.

Closes #70

## Summary The confirm-tx page displayed token contract addresses as plain text while other views (addressToken) used a styled container with color dot, copy-on-click, etherscan link, and token metadata. ## Changes - **index.html**: Updated confirm-token-section to use `bg-hover rounded-md` container with bold header, matching addressToken style - **confirmTx.js**: Render contract address with `addressDotHtml` color dot, dashed-underline copy-on-click span, etherscan link, and optional name/symbol metadata. Added click-to-copy handler. - **send.js**: Pass `tokenName` to confirmTx for richer contract info display. Import `TOKEN_BY_ADDRESS` for name resolution. ## Audit Results Audited all views for display inconsistencies: - **confirm-tx**: Token contract was plain text — now uses styled container ✅ - **addressToken**: Already uses proper styled container (reference implementation) - **approval.js**: Uses simpler address display but appropriate for dapp approval context - **txStatus.js**: Uses address display consistent with its context - **transactionDetail.js**: Uses well pattern for calldata display, consistent No other contract address display inconsistencies found that warrant changes in this PR. Closes #70

sneak was assigned by clawbot

2026-02-28 21:06:42 +01:00

clawbot added 1 commit 2026-02-28 21:06:43 +01:00

fix: consistent token contract display on confirm-tx page

check / check (push) Successful in 23s

Details

bc4e78c466

The confirm-tx page displayed token contract addresses as plain text,
while addressToken used a styled container (bg-hover rounded-md) with
color dot, copy-on-click, etherscan link, and token metadata.

Changes:
- confirm-tx HTML: use bg-hover rounded-md container matching addressToken
- confirm-tx JS: render contract with addressDotHtml, copy-on-click span,
  etherscan link, and optional token name/symbol metadata
- send.js: pass tokenName to confirmTx for richer contract display
- Add click-to-copy handler for contract address on confirm-tx

Closes #70

clawbot commented

2026-02-28 21:06:51 +01:00

Closing as duplicate of #73.

clawbot closed this pull request

2026-02-28 21:06:52 +01:00

clawbot referenced this pull request

2026-02-28 21:06:53 +01:00

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