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

fix: align receive view address display with addressDetail pattern #71

Closed
clawbot wants to merge 1 commits from fix/issue-58-receive-address-consistency into main
pull from: fix/issue-58-receive-address-consistency
merge into: :main
Conversation 2 Commits 1 Files Changed 2
+22 -11
clawbot commented 2026-02-28 20:59:55 +01:00
Collaborator
Copy Link
Copy Source

The receive view used formatAddressHtml() which rendered addresses with a different structure (bold title label above address) than addressDetail's pattern (separate dot + full address + etherscan link inline). This violated the display consistency policy.

Now uses the same element structure as addressDetail:

  • Separate receive-address-dot, receive-address-full, and receive-etherscan-link spans
  • ENS name shown in a separate element matching addressDetail's address-ens pattern
  • title="Click to copy" attribute on container for discoverability

docker build . passes.

Closes #58

The receive view used `formatAddressHtml()` which rendered addresses with a different structure (bold title label above address) than addressDetail's pattern (separate dot + full address + etherscan link inline). This violated the display consistency policy. Now uses the same element structure as addressDetail: - Separate `receive-address-dot`, `receive-address-full`, and `receive-etherscan-link` spans - ENS name shown in a separate element matching addressDetail's `address-ens` pattern - `title="Click to copy"` attribute on container for discoverability `docker build .` passes. Closes #58
clawbot added 1 commit 2026-02-28 20:59:55 +01:00
fix: align receive view address display with addressDetail pattern
check / check (push) Successful in 22s
Details
b9c3d99fed 
The receive view was using formatAddressHtml() which rendered addresses
with a different structure (bold title label above address) compared to
addressDetail's pattern (separate dot + full address + etherscan link
inline elements). This violated the display consistency policy.

Now uses the same element structure as addressDetail: separate
receive-address-dot, receive-address-full, and receive-etherscan-link
spans, with ENS name shown in a separate element matching addressDetail.

Closes #58
clawbot added the needs-review label 2026-02-28 21:00:07 +01:00
clawbot commented 2026-02-28 21:00:14 +01:00
Author
Collaborator
Copy Link
Copy Source

Aligned the receive view address display with the addressDetail pattern:

  • Replaced formatAddressHtml() with separate receive-address-dot, receive-address-full, and receive-etherscan-link elements matching addressDetail's structure
  • Added receive-ens element for ENS name display, matching addressDetail's address-ens pattern
  • Added title="Click to copy" on the container div

docker build . passes. Ready for review.

Aligned the receive view address display with the addressDetail pattern: - Replaced `formatAddressHtml()` with separate `receive-address-dot`, `receive-address-full`, and `receive-etherscan-link` elements matching addressDetail's structure - Added `receive-ens` element for ENS name display, matching addressDetail's `address-ens` pattern - Added `title="Click to copy"` on the container div `docker build .` passes. Ready for review.
clawbot commented 2026-02-28 21:00:19 +01:00
Author
Collaborator
Copy Link
Copy Source

Closing — #58 was already fixed by PR #69 (merged). This is a duplicate.

Closing — #58 was already fixed by PR #69 (merged). This is a duplicate.
clawbot closed this pull request 2026-02-28 21:00:20 +01:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
No labels needs-review
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
clawbot sneak (Jeffrey Paul)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/AutistMask#71