search

Uniswap / interface

🦄 Open source interfaces for the Uniswap protocol
https://app.uniswap.org
GNU General Public License v3.0
4.95k stars 5.02k forks source link

Full Range 'I Understand' box should have reject option #2570

Open willhennessy opened 3 years ago

willhennessy commented 3 years ago

Steps

  1. Add Liquidity page
  2. Click 'Full Range'
  3. Read the warning message and decide that you agree, you probably shouldn't use Full Range

Current: the only way to get out of this flow is to click 'I understand'. That changes the price range to zero to infinity, which is not desired and is a bit tricky to adjust back to a normal range.

Suggested behavior: the 'I understand' dialog box should include an option like 'nevermind', 'decline', or an 'X' button

Only P2

Screen Shot 2021-10-12 at 11 33 17 PM
callil commented 3 years ago

Can we kill the button? I think a warning is sufficient and should just disappear when you re-adjust the range,