Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

address copy button added #5083

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ijlal321
Copy link
Contributor

Feature Added: Copy Address Button in Accounts Dropdown

Reason:
Previously, to copy an address, users needed to select the address, copy it, and then reselect the previous address. The goal of this feature is to streamline the process by providing a button that allows users to directly copy the address, eliminating the extra steps.

Implementation Details:
Originally this dropdown uses a html native select (which does not support any buttons inside option ). This is now replaced by react-select component. React-select allows a dropdown option to have buttons.

Code Dependability:
This new select is made to look exactly as previous one. Most classes and components used are same as previous select. The copy button is imported from remix-copy. All fonts are working perfectly.

Copy link

netlify bot commented Aug 16, 2024

👷 Deploy request for remixproject pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 8cc95a3

@ijlal321 ijlal321 marked this pull request as ready for review August 16, 2024 20:55
@Aniket-Engg
Copy link
Collaborator

@ijlal321 linting is failing for the PR

@ijlal321
Copy link
Contributor Author

@ijlal321 linting is failing for the PR

Got it, fixed it 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants