Loopring / loopring-wallet-feedback

钱包产品需求汇总(包括安卓和iOS)
10 stars 3 forks source link

Fee selection in "Mint NFT" will default to ETH after a refresh, and won't properly tell the user if the funds are insufficient #172

Open sk33z3r opened 2 years ago

sk33z3r commented 2 years ago

System:

Repro:

  1. Use WalletConnect to connect and unlock your wallet on loopring.io
  2. Go to L2 Wallet > My NFT
  3. Click "Mint NFT" in top-right OBSERVE: "Mint NFT" modal appears, the Fee section gives an amount and is set to LRC
  4. Click the Fee drop down and select ETH as the token OBSERVE: ETH gets selected, and I am presented with an insufficient funds message (because I have 0 ETH in L2)
  5. Select LRC again OBSERVE: LRC gets selected, and the insufficient funds error disappears as expected
  6. Click the "X" in top-right to close the modal
  7. Refresh the page
  8. Click "Mint NFT" in top-right again

Expected: Modal looks mostly the same, with LRC selected as the token since it was the last one I had selected. Fee amount might be slightly different due to changing gas prices.

Actual: Modal appears, but the fee token is now set to ETH and there is no error telling me that I have insufficient funds.

  1. Click the fee dropdown again and select any other token

OBSERVE: This seems to reset the issue. If you select a token you have a 0 balance of, then the insufficient funds message appears as expected. Now clicking back on ETH will give the proper insufficient funds notice.

Here is a gif of this occurring:

gif_20220227_113521