Crypto-to-fiat: Drains history table UI fixes

JoinColony / colonyCDapp

An iteration of the Colony Dapp sporting both a fully decentralized operating mode, as well as a mode enhanced by a metadata caching layer

5 stars 14 forks source link

Crypto-to-fiat: Drains history table UI fixes #2662

Open melyndav opened 3 months ago

melyndav commented 3 months ago

Description

This issue covers the transaction table that requires some minor adjustments to align it with other table styling. I haven't been able to test the default state yet, so more issues my come with this. .

[ ] The status arrow should show by default, it's currently not showing, and the user do not know it's available for sorting. The table should stay by default sorted by date, so no change here.

[ ] Mobile table styling should match Figma designs on mobile

Current Screenshot 2024-07-29 at 12 24 20 Figma Screenshot 2024-07-29 at 13 57 19

rumzledz commented 3 months ago

@melyndav number 6 might be problematic because I don't think our tables are configured to show an arrow for the column that's not responsible for the current sort order

jakubcolony commented 2 months ago

Hi @melyndav, I edited the issue to remove the following points that have already been resolved by other PRs:

[x] Amount column styling needs to match Figma with a weight and colour change.

[x] Date column text should match Figma.

[x] Status pills font should match Figma (these are existing components, so these can be pulled from other tables).

[x] Can we add a text-blue-400 hover state to the external links?

melyndav commented 2 months ago

Thanks for updating @jakubcolony 👍

rdig commented 1 day ago

@Nortsova is this similar to #2695 ? If so, it might be a good idea to increase the scope here, and fix all tables, not just the one in C2F