vegaprotocol / frontend-monorepo

Toolkit for building apps that interact with Vega
https://vega.xyz
MIT License
25 stars 17 forks source link

New markets page snags #6316

Closed JonRay15 closed 5 months ago

JonRay15 commented 5 months ago

The Chore

Feedback for further improvements to the markets page

Sorting issue

DP issues

Column width

I have lost the edge of the "open interest" column on a 13in monitor with 90% zoom on Chrome. If I go to 100% I lose it almost completely. Therefore in both cases I have a horizontal scroll bar.

image

Column widths on Proposed & Closed

image

Spacing / padding

image

24h change section

It seems like the spark line graph doesnt fill the entire height of the row? The chart starts from maybe a third of the way up? But then it goes to the very top?

image

I'm conscious this is a share component with the market dropdown so anythign we do here will impact that too.

image

Its particularly jarring on things in auction where they get this weird grey bar.

image

Not sure exactly is the best course of action ... I can think of two:

  1. Fill the full row top to bottom, but given we have the fill under the line that will mean a lot of filled in space. We would likely need to cut it off somewhere so maybe we have a margin top and bottom but lets have the same margin since at the moment it is only really occupying the top 2/3 of the row
  2. If we think that is going to lead to too much filled space, could we do that but not have the gradient fill under the line? It seems to me that it is that gradient - and trying to work out where it stops - that is causing the issue here? If I look at other examples most dont have that gradient under the line so dont face this problem

image

Additional details / background info

bwallacee commented 5 months ago

Alignment here on the price change can be improved. image (31)

daunatv commented 5 months ago

Assume this is the issue you are referring to in terms of sorting, where in the example SOL should end up on top considering the volume.

https://github.com/vegaprotocol/frontend-monorepo/assets/100769129/1f404ad2-0e4a-4fed-ae6f-aa8613680c86

witgaw commented 5 months ago

I'd be nice to always have two decimal places for 24h notional (grey numbers below 24h volume figure) to easily tell the difference between markets. image

JonRay15 commented 5 months ago

I am bumping this up the list because the sorting bug is still there .. and the DP issue is still there.

I also think we should simply show ZERO DP on the $ values ... If there is $411,031.45 of volume I absolutely do not care about the 45c!