pendulum-chain / portal

The Pendulum/Amplitude portal UI
GNU General Public License v3.0
1 stars 1 forks source link

Text changes on the Nabla swap screen #473

Closed prayagd closed 4 weeks ago

prayagd commented 1 month ago

Context

This ticket is to fix inconsistencies across the swap UI

Requirement

Reference

Screenshot 2024-05-28 at 6 27 08 PM
TorstenStueber commented 1 month ago

I added "your" on purpose. Before that it was never clear in the UI what balances and amounts apply to you or to the pool in general (e.g., is this a pool balance).

The UI is full of different numbers and I feel this is quite overwhelming to users (particularly if they are not numbers affine).

I suggest that we make it dead obvious what the numbers mean, e.g.:

There are so many ways what a number can mean on the UI and we should not make the user guess.

prayagd commented 1 month ago
Screenshot 2024-05-29 at 11 53 24 AM

@TorstenStueber above is the screenshot of uniswap screen, leading DEX in the entire crypto ecosystem. Do you find this confusing too? The attempt here is to make familiar to the general DEX UI.

I added "your" on purpose. Before that it was never clear in the UI what balances and amounts apply to you or to the pool in general (e.g., is this a pool balance).

Sorry dont agree, i dont think so any user on the swap screen would confuse the balance as pool balance.

TorstenStueber commented 1 month ago

Yes, I find this confusing. Why would it be clear that this is not the pool balance, which is equally reasonable?

If every new project just copies what already exists, disruption would never be possible. But that's just my opinion. Happy to support a change if this a fringe opinion.