Add copy to clipboard#497
Open
iyanumajekodunmi756 wants to merge 3 commits into
Open
Conversation
Expose a screen-reader-only label mode so placeholder-driven inputs can still meet WCAG 2.1 AA programmatic naming requirements via htmlFor/id. Closes CredenceOrg#381 Co-authored-by: Cursor <cursoragent@cursor.com>
- Create AddressDisplay component with truncation, hover/focus full address, and copy button - Use AddressDisplay in Dashboard for wallet address - Use AddressDisplay in Transactions for transaction hashes - Add confirmation toast on copy Closes CredenceOrg#383
|
@iyanumajekodunmi756 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Truncate display, full address on hover/focus, and confirmation toast on copy.
Background
This change improves the day-to-day experience for the people who consume this code (operators, downstream contracts, frontend engineers, support). It is not a strict bug, but the current behaviour forces workarounds and the proposed change removes them.
Acceptance criteria
The change matches the summary above.
No regression in the existing test suite.
The change is documented where it is observable (README, docs/, public API reference).
Lint, type-check, and tests all pass locally.
PR description references this issue with Closes #.
Implementation hints
Keep the public API surface backwards-compatible if possible. If a breaking change is unavoidable, document the migration step.
Land any new constant in a single place (config/, consts.rs, or the equivalent) and re-use it everywhere.
Update the relevant docs/ page in the same PR so reviewers see the docs change together with the code change.
Add or update the corresponding test so the new behaviour is locked in.
Repo-specific notes
Run npm run lint, npm run build, and the unit tests locally before pushing.
If you change a public component prop, update the matching Storybook story and the docs/COMPONENTS.md entry.
Respect the design tokens (CSS variables / Tailwind config); do not hard-code colours, spacing, or radii.
Out of scope
Unrelated refactors in adjacent files.
Stylistic-only changes (formatting, renaming) that are not required by the fix.
Anything beyond the acceptance criteria above; surface follow-ups as separate issues.
How to claim and submit
Comment on this issue saying you'd like to take it on; wait for a maintainer to assign you (avoids duplicated effort).
Open a PR that references this issue (Closes #).
Make sure CI is green and request review from a CODEOWNERS maintainer.
PRs that close this issue and pass review may qualify for a reward — see the MAYBE REWARDED label and the GrantFox OSS campaign page.
closes #383