-
Notifications
You must be signed in to change notification settings - Fork 5
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
Added a dropdown for wallets and changed UI #7
Conversation
PR is made to change the UI as mentioned in the following issue:- #3 |
Remove the .yarn folder in this PR |
Done |
You have modified the wallet-adapter-ui |
You have modified the wallet-adapter-ui |
All you need is to focus on the useWallet() hook and the wallets object is going to contain the details |
Okay I will work on this and get back to you as soon as I am done. |
Like you said I removed the WalletMultiButton and removed styling from css module file. Please tell me if there are anymore changes that need to be done. |
@0xPratik I added support for all wallets, updated the UI as you said, added a dropdown menu and removed the WalletMultiButton. Everything looks fine on my end. Please check if everything is okay and let me know if I need to fix anything. |
Fine. I will open a new PR and try to fix all these issues. |
Note:- Detects and lists all wallets installed but currently connects only to Phantom.