warn popup updated

[maria-vslvn]

Closes #672
Cancel popup updated according to new mockups

[github-actions]

• 🔭 IDO UI: Easy IDO auction

[elena-zh]

@maria-vslvn ,

1. additional message on the pop-up should not be displayed when bidding tokens are not ETH (for Rinkeby and Mainnet networks) and xDAI (for xDAI network)
   

2. Token pair (DAI per GNO) should not me mocked on the pop-up and should consider < selling token > and < buying token > values
   

3. Label should not contain 'Min (max)' in 1 line. Min should be displayed when label is 'Min bidding price' in the price field. Max should be displayed when label is 'Max bidding price' in the price field. See my comment here
   

4. Warning message should not be mocked as well. It should take into consideration bidding token value:

• If you Cancel an order in xDAI, you will receive back WXDAI tokens in xDAI. - for xDAI network
• If you Cancel an order in ETH, you will receive back WETH tokens in Rinkeby.- for Rinkeby network
• If you Cancel an order in ETH, you will receive back WETH tokens in Mainnet.- for Mainnet network

[maria-vslvn]

@elena-zh please check

[elena-zh]

@maria-vslvn ,

1. The 2nd warning message on the cancellation pop-up should be always displayed in all auctions:
   

2. Besides, neither the 1st nor the 2nd warning message are not displayed on the cancellation pop-up when cancel orders with ETH/XDAI bidding token

3. Min 'ETH' instead of 'WETH' (and xDAI instead of WXDAI) should be displayed on the pop-up message
   

4. Token's pair should be reverted as well (besides changing min/max label):
   

5. Token icons should be bigger and should not be shifted towards numbers (see as an example 'Place order' pop-up)
   

[maria-vslvn]

updated

[elena-zh]

Hi @maria-vslvn ,
changes look much better, however, these issues are still not fixed:

1. When invert price, not only token's pair should be changed, price itself should be changed as well
   

2. Still, i do not see an additional warning neither for ETH token in Rinkeby and Mainnet, nor for XDAI token in the XDAI
   
   

[maria-vslvn]

@elena-zh please check

[elena-zh]

@maria-vslvn , great job!

The only thing I'd like to notice, that token names on the modal are WETH and wxDAI.

However, I'm not sure whether it is an issue.. @josojo , could you please take a look and clarify whether we should show ETH/xDAI everywhere on the modal, or leave it as it is?

Also, I did not have a possibility to test changes in the Mainnet.. Alex, could you please check these changes there?

Thanks

[josojo]

Yes the behaviour would be correct with ETH/xdai instead of WETH/wxdai.

@maria-vslvn you can just use the function:

getTokenDisplay(biddingToken, chainId) or getTokenDisplay(auctioningToken, chainId)

to display it correctly

[maria-vslvn]

@elena-zh please check

[josojo]

NIce. thanks. Looks good now

[elena-zh]

Additional warning now is missing when cancel an order in ETH:

[josojo]

@maria-vslvn could you please tackle the issue from elena? then we can merge it and complete the task

[maria-vslvn]

@elena-zh please check

[elena-zh]

@maria-vslvn , still there is no additional message =(