fix: add time elapsed in transaction list (Home)

hyperlane-xyz / hyperlane-explorer

Web app for the Hyperlane Explorer

https://explorer.hyperlane.xyz

Other

50 stars 46 forks source link

fix: add time elapsed in transaction list (Home) #71

Open tiendn opened 2 months ago

tiendn commented 2 months ago

Issues: UX performance about transaction list in page /home

We should have a time elapsed in transaction list (Home Screen) for better UX.

Solutions:

Add a TimeElapsed.tsx component with setInterval each second for checking new value of readable time string
Replace current function string to TimeElapsed component only on Home screen

https://github.com/hyperlane-xyz/hyperlane-explorer/assets/15717476/39d7fcdf-6cdb-40ce-85ce-acec3ee3b2e7

vercel[bot] commented 2 months ago

Someone is attempting to deploy a commit to the Abacus Works Team on Vercel.

A member of the Team first needs to authorize it.

jmrossy commented 2 months ago

Thanks for this contribution but it was intentional to keep the time sent static. We believe it's too much noise when all the numbers change constantly. But we'll consider your change and get back to you.

tiendn commented 2 months ago

Can you tell me why the time is kept static? I think the animation, when a new transaction is coming, blurs and replaces, making it difficult to follow.