connext / monorepo

Connext is a modular stack for trust-minimized, generalized communication between blockchains.
https://docs.connext.network
MIT License
292 stars 172 forks source link

[UI] Slippage warning doesn't show up without expanding the dropdown #5352

Closed Tommy-R8 closed 8 months ago

Tommy-R8 commented 9 months ago

Problem

If a transaction's estimate exceeds slippage, it should show a warning. The problem is that this warning is only visible if you expand the "Additional Details" dropdown to modify the slippage tolerance. The warning should be clearly visible from the main screen. image image

Impact

If users don't see the warning they will get their transaction stuck

Proposed Solution

Show the warning on the main screen without needing to expand the menu

Acceptance Criteria

The user should instantly and clearly see in the main UI if their tx won't succeed because the estimated slippage is higher than the tolerance

pjenviri commented 9 months ago

Updated and deployed.
PR: https://github.com/CoinHippo-Labs/connext-bridge/pull/271