Fujicracy / fuji-v2

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

UI issues #721

Closed NikolaiYurchenko closed 8 months ago

vercel[bot] commented 8 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fuji-v2-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 21, 2023 11:38am
ferostabio commented 8 months ago

@NikolaiYurchenko I was playing around with the work you did, and I think everything we're doing is actually worst. Now... I realized that the underlying issue is that we have one MarketsTable with two different data sources. Let's just have two tables and show one of them according to the selected tab! No transition will be needed (tableData will have to be an array containing both sets of data)

ferostabio commented 8 months ago

Since you're at it, please have a look at this two things:

ferostabio commented 8 months ago

Thanks for the fixes @NikolaiYurchenko 😁

The glitch is still there, though. Wasn't expecting that, since they are now different tables. I would say...

1 - Try to see if there is a way out of it. They are different tables after all now. 2 - If you find no way out, do implement the loader... but not just one row. We have the row count for each list, so we can now how many skeleton rows to show (and for a bit less time)

ferostabio commented 8 months ago

Another small UI issue @NikolaiYurchenko ! Left panel shouldn't change width when the "...after..." component appears.

https://github.com/Fujicracy/fuji-v2/assets/501915/878295e0-25c8-48c2-861d-49a144af1cec

NikolaiYurchenko commented 8 months ago

Thanks for the fixes @NikolaiYurchenko 😁

The glitch is still there, though. Wasn't expecting that, since they are now different tables. I would say...

1 - Try to see if there is a way out of it. They are different tables after all now. 2 - If you find no way out, do implement the loader... but not just one row. We have the row count for each list, so we can now how many skeleton rows to show (and for a bit less time)

well, it will be blinking just because it is insta-changing, moreover on a dark background

ferostabio commented 8 months ago

well, it will be blinking just because it is insta-changing, moreover on a dark background

If you play the video in slow motion the issue is super clear 😅 (do vlc at .25x). For a brief moment, while switching tabs, you get lending data on the borrowing table and borrowing data on the lending one.

Have a look at the attached screenshot. Have I switched from borrowing to lending, and while its going to show the right info, you can see here that its showing borrowing data. Since we move this data around, its really visible and it produces the glitch.

1 - Switched to lending, showing borrowing data

Captura de pantalla 2023-08-17 a la(s) 15 04 36

2 - Finally shows lending data

Captura de pantalla 2023-08-17 a la(s) 15 06 21

3 - Same thing happens going back to borrowing

ferostabio commented 8 months ago

We are fixing a couple of minor UI issues here, but want your feedback about something @Markoyw !

If you visit the production app, when you switch from borrowing to lending in the markets page, the experience is not great. Same thing happens in the positions page, to a lesser degree -there is a data missmatch, for a brief moment, and it feels a bit weird. We tried a couple solutions, and the best thing we came up with is showing a loader to smooth the transition.

Please visit this link to have a look.

Agree? Disagree? Ideas?

Markoyw commented 8 months ago

Agree, I like it a lot!