web icon indicating copy to clipboard operation
web copied to clipboard

Thorchain LP: Add Liquidity to an existing position

Open 0xean opened this issue 1 year ago • 4 comments

AC

  • figma
  • for users already have an existing liquidity position in the same pairing that they are attempting to add to, show the existing position and make it clear they are adding vs creating a new position
  • remove existing warning for non supported positions since now have this view.

0xean avatar Apr 12 '24 16:04 0xean

image

kaladinlight avatar Jul 02 '24 21:07 kaladinlight

@reallybeard to work on some mocks

0xean avatar Jul 03 '24 22:07 0xean

https://www.figma.com/proto/hq8z2S8QwL2DciuGR8b0GR/Thorchain-LPs?node-id=1279-16715&t=9VtNEDllB1f7Anjb-0&scaling=min-zoom&content-scaling=fixed&page-id=1279%3A12227&starting-point-node-id=1279%3A13508

0xean avatar Jul 08 '24 17:07 0xean

Screenshot 2024-07-08 at 4 18 50 PM

grey out the disabled button and add tooltip to explain that this action is not supported by TC

cc @twblack88

0xean avatar Jul 08 '24 22:07 0xean

We probably want tooltips for the new positions too (unless we already have a warning modal)

Not supported copy: "Due to your existing Asymmetric Rune position, depositing into this pool with both assets isn't possible. Please withdraw your current position if you wish to create a new LP with both assets."

New Position copy: This will create a new LP position instead of adding to the existing one.

twblack88 avatar Jul 09 '24 17:07 twblack88