Open VolodLytvynenko opened 3 days ago
@VolodLytvynenko
Currently we explicitly specify the priority of tokens the following way:
{"SNT" 1 "STT" 1 "ETH" 2 "DAI" 3}
It means those four tokens will always be on top, no matter the balance. The rest will be sorted based on balance. Do you want me to change that?
Hi, yes, it would be better to fix it. However, the priority of this issue is minor, so I’ve removed it from 2.32.
Here are some reasons why this behavior should work as it is:
This is clearly aligned with the design. Even if SNT is grouped with some other assets, it’s displayed lower than others, and this approach is similar to other apps like Paraswap, which we use as a provider for swapping.
On the wallet main page, the sorting logic is based on fiat balance in descending order (screen) and SNT, DAI, and ETH follow this sorting as well. For consistency, it would be better to maintain the same approach across all screens. Currently, SNT, DAI, and ETH are prioritized on some pages, while on the wallet main page, sorting is based on fiat balance in descending order, which is inconsistent.
In fact, SNT, ETH, and DAI are not exceptions. Sometimes, other assets are also not shown in descending order. Therefore, this feature might need to be re-implemented to ensure consistency.
Such behavior will align with the desktop
Steps:
Actual result:
Assets not sorted by fiat available balance in descending order
Expected result:
Assets should be sorted by fiat available balance in descending order on "assets to pay" and "assets to receive" screens https://www.figma.com/design/AD2JSKg0I8dZcylyiGa62O/Swap-for-Mobile?node-id=43-170626&node-type=frame&t=M5nVcC6Sxn3Nuue6-0
OS:
IOS, Android
Devices:
ENV:
Nightly 18 Nov, 2024