-
Notifications
You must be signed in to change notification settings - Fork 1
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
Don't show empty balances when account not connected #477
Don't show empty balances when account not connected #477
Conversation
✅ Deploy Preview for rococo-souffle-a625f5 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good @ebma with the swap UI, can we remove the "0" also from the backstop pool UI? and then go ahead and merge.
I left the 'My pool balance: 0' at the backstop pool because I thought it might look weird because it's so empty otherwise. But I can remove that too, no problem. |
Please check again @prayagd |
@ebma the text "My pool balance" also went away, no worries lets merge. This is for the testnet campaign anyway |
Wait, I thought that's how you wanted it to look like? 😅 At least that's exactly what I changed it to. Did you expect something else? I can also change it again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ebma perfect ✅
@ebma oh sorry for the confusion. just show this text back, that's it |
Out of the three options we have:
Isn't option 3) the most confusing? 😅 |
If there is nothing the user can do on that page, without connecting their wallet, then that's where we should be sending them (and so should be what they see). References from other DEXes HydraDX's Wallet Page: Pablo's portfolio view Both dApps:
Specifically on the options:
Since my suggested solution might require some more work, can the immediate solution just be that the animation is made static, everything else remaining the same? See HydraDX's wallet view without connecting wallet for reference. |
I suggest: remove "My pool balance" as @ebma did and just add a "Connect Wallet" button instead. Don't we have a component for this and adding such a button is a trivial change? |
Thanks for providing some references. Personally, I think it's fine the way it is now with not showing anything related to balances as long as no wallet is connected. I can add the change @TorstenStueber suggested and instead make the button for depositing/swapping prompt to connect a wallet account instead. The same is already done when you go to the Spacewalk issue/redeem dialog without being connected to a wallet. |
Sounds good, so both buttons become Connect Wallet, when the user connects wallet, the My Pool Balance appears with the correct amount. |
I changed it so that all Nabla primary action buttons are replaced with 'Connect to Wallet'. And I added the change to make the wallet account selection scrollable. |
@ebma looks good we can merge, checked all the buttons and all the connect wallet buttons work and open the pop-up |
I replaced all related loaders I could find with just 'nothing' expect for the Backstop Pool screen where 'My pool balance' now shows
0
if no account is connected.Closes #469.