[dxfareed]

Closes #4

What changed: Gave an accessible name to the 15 icon-only buttons in the wallet that had none. 18 added lines, all ARIA attributes, across 10 files. Nothing removed.

Why: These announced as a bare "button" to a screen reader. Seven had a title tooltip, which is unreliable as an accessible name and invisible on touch; eight had neither.

control	label
dashboard copy address	Copy wallet address
dashboard fee-payer notice dismiss	Dismiss fee-payer warning
dashboard stranded-balance dismiss	Dismiss stranded balance notice
NFT list refresh	Refresh NFTs
NFT detail close	Close NFT details
agent clear history	Clear history
agent back	Back
contacts delete	Delete contact
multisig remove signer	Remove signer
install banner dismiss	Dismiss install prompt
contact picker close	Close contact picker
settings/agent back, settings/profile back	Back to settings
token/[code] back	Back
settings/agent enable toggle	Enable AI agent

Three notes on the choices:

• Labels reuse vocabulary already in the tree. Back to settings appears 4× and Back once, so the back arrows follow that rather than a new phrasing.
• The two dashboard dismiss buttons were both titled Dismiss. They sit on the same screen, so identical names are ambiguous out of context; their aria-labels distinguish them while the visible tooltips stay as they were.
• The settings/agent toggle also gets role="switch" and aria-checked, matching the Toggle in components/ui/primitives.tsx, which is the existing house pattern for a switch. Without a role it announced as a button with no state. type="button" came along from that same pattern; it is a no-op here, there is no <form> in the file.

What I deliberately left alone: buttons whose accessible name is already rendered text. I scanned all 228 <button> elements under app/ and components/ and classified them; the ones that look icon-only but are not (nfts dev-mode toggles, the Copy/Copied button, NetworkSwitcher) all render a visible label. The settings gear in TabBar/AppShell is likewise already correct: visible text label with aria-hidden on the glyph.

How tested:

• npm run typecheck in frontend/wallet clean.
• npm run build succeeds, all routes compile.
• npm test: 251 passed. One suite, feeBump.test.ts, fails to load on ReferenceError: TextEncoder is not defined. That is pre-existing on main and unrelated to this change; it is fixed separately in test(wallet): cover truncateAddress edge cases #11.
• Re-ran my scan after the change: zero icon-only buttons without an accessible name remain, down from 15. No element ended up with a duplicate aria-label.
• Diff is purely additive with no style, layout or copy touched, which is what satisfies "no visual change" structurally.

Not tested: I did not verify with a screen reader or the a11y devtools pane, so that acceptance box is unticked from my side. There is no eslint-plugin-jsx-a11y in the project and no component-render test setup (@testing-library/* is not installed), so I had no way to assert computed accessible names automatically without adding a dependency, which felt out of scope here. Worth a real screen-reader pass before merge, particularly on the role="switch" toggle.

Notes:

npm run lint in frontend/wallet is broken on main, unrelated to this change: the script is next lint, which Next 16 removed, so it fails with Invalid project directory provided, no such directory: .../lint. There is also no eslint config in that package. Happy to send a follow-up migrating it to eslint directly, and adding eslint-plugin-jsx-a11y would make regressions like this issue fail CI rather than needing a manual audit.

Note also that frontend/wallet's jest suite is not run by CI at all (only the sdk and mobile jobs invoke tests), so nothing here is currently enforced automatically.