-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: improve Orderbook UI #556
Conversation
This reverts commit d31a947.
Deploy PreviewsTestnetPreview URL: https://d31a9476-d1afd1a6--duality-xyz-testnet.netlify.app BetaPreview URL: https://d31a9476-d1afd1a6--duality-xyz-beta.netlify.app |
Deploy PreviewsTestnetPreview URL: https://e4199fc9-5015a777--duality-xyz-testnet.netlify.app BetaPreview URL: https://e4199fc9-5015a777--duality-xyz-beta.netlify.app |
e4199fc
to
e4712bb
Compare
Deploy PreviewsTestnetPreview URL: https://d9f99edc-a8a0539b--duality-xyz-testnet.netlify.app BetaPreview URL: https://d9f99edc-a8a0539b--duality-xyz-beta.netlify.app |
Deploy PreviewsTestnetPreview URL: https://46aae64d-b63d1948--duality-xyz-testnet.netlify.app BetaPreview URL: https://46aae64d-b63d1948--duality-xyz-beta.netlify.app |
Deploy PreviewsTestnetPreview URL: https://b612bc8a-79591317--duality-xyz-testnet.netlify.app BetaPreview URL: https://b612bc8a-79591317--duality-xyz-beta.netlify.app |
- .toPrecision can return scientitific notation strings which will not pass validation on some numeric input fields - not all need to be changed but its best to change all anyway
Deploy PreviewsTestnetPreview URL: https://b5e7868d-8f2bdeb5--duality-xyz-testnet.netlify.app BetaPreview URL: https://b5e7868d-8f2bdeb5--duality-xyz-beta.netlify.app |
- null parameter means to return an empty data set - a non-null parameter means to request a data set - hopefully this is clearer
- not just GOOD_TIL_TIME but also GOOD_TIL_CANCELLED
- in the Orderbook mostly display tokens are expected
Deploy PreviewsTestnetPreview URL: https://85919acf-61838b52--duality-xyz-testnet.netlify.app BetaPreview URL: https://85919acf-61838b52--duality-xyz-beta.netlify.app |
## [2.11.0](v2.10.0...v2.11.0) (2024-06-05) ### Features * improve orderbook order card ([#556](#556)) ([45ad9c9](45ad9c9))
🎉 This PR is included in version 2.11.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This PR aims to improve the Orderbook experience based on several user interview and feedback sessions
it includes several Orderbook changes:
LimitOrder card changes:
amountIn
oramountOut
input fieldsTokenInputGroup
)Chart component changes:
fix
: improve/reduce re-rendering of Chart on page start/loadfix
: remove manual addition of "spacing" bars, activate option for automatic "spacing" barsnew Chart-Depth connection component:
Depth tab/table changes:
fix
: puts higher prices at top of table (to align to y-axis of chart)Recent Trades List tab/table component changes:
My Orders table components changes:
and several general fixes to enable these updates:
useBuckets
useRealtimePrice
hooksuseRealtimeBasePrice
anduseRealtimeDisplayPrice
isLoading
state of react-query statesisLoading
of the SWR state now maps to the react-queryisLoading
state as should make senseonAccumulated
callback to receive the latest data updates as well as the accumulated stateaddress
field (resulting in querying all order txs).reverse()
mutations (which should be avoided in React renders)and several updates to components to enable these updates:
header
andsubheader
propsinactive
proponChange
to be optionalonChange
is optional