Fujicracy / fuji-v2

Cross-chain money market aggregator
https://fuji-v2-frontend.vercel.app
15 stars 10 forks source link

UI issues v4 #497

Closed NikolaiYurchenko closed 1 year ago

NikolaiYurchenko commented 1 year ago

closes https://github.com/Fujicracy/fuji-v2/issues/496 figma link for reference

github-actions[bot] commented 1 year ago

Please test this PR on: https://v2-staging-497-dot-fuji-306908.ey.r.appspot.com

ferostabio commented 1 year ago

Banner’s closing button should be an “X”, not “Dismiss”. Banners can have an action integrated in the message, and that’s going to be underlined like the Dismiss button is.

https://user-images.githubusercontent.com/501915/235749360-3e3b7d55-a859-40c3-9aaa-55207df3255a.mov

ferostabio commented 1 year ago

Markets show the first element as the best one before loading the data, so we show the wrong element as the best for a couple of seconds (notice in the video that we show Optimism as the best one until loading the data and then Gnosis is the best)

https://user-images.githubusercontent.com/501915/235750906-0677219f-e11e-4af3-910c-34fa5c4e9d23.mov

Also, increase the "Best" button's right padding to match the left one.

ferostabio commented 1 year ago

Hovering on the new liquidation price tooltip shouldn't change the pointer.

ferostabio commented 1 year ago
ferostabio commented 1 year ago

Yield modal's UI messed up, have a look @NikolaiYurchenko

Captura de pantalla 2023-05-03 a la(s) 18 03 58