Full Range 'I Understand' box should have reject option
Open
@maggievalentine is already working on this.
Since Oct 13, 2021.
design
improvement
p2
- Dominant language
- TypeScript
- Stars
- 5.5k
- Forks
- 5.4k
- PR merge metrics
- No merged PRs in 30d
Description
Steps
- Add Liquidity page
- Click 'Full Range'
- 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
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.