999/2 decimals percentage

[alfetopito]

Summary

Closes #999
Closes #1148

2 decimals precision for price impact

To Test

1. Fill in all inputs while on Mainnet or xDai

• Observe price impact is limited to 2 decimals precision
• When percentage < 0.01, it shows < 0.01
• It does not display insignificant zeros: 0.90 => 0.9

[github-actions]

• 🔭 GP Swap: Gnosis Protocol v2 Swap UI

[anxolin]

Would these also be affected?

• cowswap/src/custom/components/CurrencyInputPanel/FiatValue/FiatValueMod.tsx

  Line 46 in ff8ae7c

  (<Trans>{priceImpact.multiply(-1).toSignificant(3)}%</Trans>)

• cowswap/src/custom/components/swap/TradeSummary/index.tsx

  Line 69 in 2f9bb89

  <ClickableText onClick={toggleSettings}>{allowedSlippage.toFixed(2)}%</ClickableText>

[elena-zh]

0% of the price impact is displayed when it's amount is less than 0.01%.

Prod:

All the rest changes LGTM!

[elena-zh]

Hey @alfetopito , we have related low priority UI issue with price impact #1148 : could you please take a look at it in this pr as well?
It might be easy to fix. :)

[W3stside]

nice!

[alfetopito]

Would these also be affected?

* https://github.com/gnosis/cowswap/blob/ff8ae7cd447262da984f1b46cbdd702bae5e3752/src/custom/components/CurrencyInputPanel/FiatValue/FiatValueMod.tsx#L46-L46

* https://github.com/gnosis/cowswap/blob/2f9bb89d5a7846a2b559be9cc840e871fb4e97da/src/custom/components/swap/TradeSummary/index.tsx#L69-L69

First one, is literally the one I had changed XD
The second wasn't part of this, but sure, updated.

[alfetopito]

0% of the price impact is displayed when it's amount is less than 0.01%.

This should probably be fixed after I merge #1281
I'll do that then try again to be sure.

[alfetopito]

Hey @alfetopito , we have related low priority UI issue with price impact #1148 : could you please take a look at it in this pr as well?
It might be easy to fix. :)

It was! Take a look now.

[elena-zh]

It was! Take a look now.

Great! Fixed! Thanks

changes made

[W3stside]

APPROVED AGAIN