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

'transaction' view has inconsistent title #65

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

sometimes it's "Transacation" and sometimes it's "Swap", but then when it's Swap it says "Action" "Swap" right below that. The main title on 'transaction' view should always be the same.

Please audit the other views for title and layout inconsistencies and file issues for any others you find.

sometimes it's "Transacation" and sometimes it's "Swap", but then when it's Swap it says "Action" "Swap" right below that. The main title on 'transaction' view should always be the same. Please audit the other views for title and layout inconsistencies and file issues for any others you find.
clawbot was assigned by sneak 2026-02-28 20:37:08 +01:00
clawbot commented 2026-02-28 20:39:08 +01:00
Collaborator
Copy Link

Fixed in PR #66.

What was changed: In transactionDetail.js, the heading was being dynamically set to tx.directionLabel (e.g. "Swap") for contract interactions. Changed to always set headingEl.textContent = "Transaction" regardless of transaction type.

Audit results: Reviewed all other views (Home, AddWallet, ImportKey, AddressDetail, AddressToken, Send, ConfirmTx, TxBroadcast, TxConfirmed, TxFailed, Receive, AddToken, Settings, DeleteWallet, ApproveTx, ApproveSign). All headings are static in the HTML and match the Screen Map in README.md. No other title inconsistencies found.

Fixed in PR #66. **What was changed:** In `transactionDetail.js`, the heading was being dynamically set to `tx.directionLabel` (e.g. "Swap") for contract interactions. Changed to always set `headingEl.textContent = "Transaction"` regardless of transaction type. **Audit results:** Reviewed all other views (Home, AddWallet, ImportKey, AddressDetail, AddressToken, Send, ConfirmTx, TxBroadcast, TxConfirmed, TxFailed, Receive, AddToken, Settings, DeleteWallet, ApproveTx, ApproveSign). All headings are static in the HTML and match the Screen Map in README.md. No other title inconsistencies found.
sneak closed this issue 2026-02-28 20:39:48 +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
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#65
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?