Skip to content
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

Enhanced trading screen sidebar #6106

Closed
12 of 25 tasks
JonRay15 opened this issue Mar 26, 2024 · 2 comments · Fixed by #6427
Closed
12 of 25 tasks

Enhanced trading screen sidebar #6106

JonRay15 opened this issue Mar 26, 2024 · 2 comments · Fixed by #6427
Assignees
Labels
branch-develop Work for the next planned release of frontend apps

Comments

@JonRay15
Copy link
Contributor

JonRay15 commented Mar 26, 2024

Story

As a user
I want a sidebar on te trade screen that makes the deal ticket more obvious and ensures I always know my asset balance
So that I dont get stuckk trying to find these things or have to navigate away from trading to find them

Acceptance Criteria

  • The minibar is removed from this page and all other pages (users will use the new portfolio screen in Portfolio page redesign #6098 if they want to manage assets)
  • The settings will need to move to the header as part of this
  • The default view on the trading screen is TRADE sketch, showing:
    • An expanded deal ticket
    • An market info tab which expands if clicked, causing the deal ticket to shrink
    • A mini-asset view which expands if clicked, causing the deal ticket to shrink
    • A datanode health indicator which launches the existing datanode selector if clicked
  • The mini asset view should show:
    • The asset for the current market
    • The icon from the icon store
    • Your balance of that asset
    • How much is allocated across any margin accounts or spot swap orders
    • How much is available collateral
    • A 3-dot menu to quick launch deposit / transfer / withdrawal actions
    • If those buttons are pressed it sends you to the portfolio page and opens the relevant tab
    • If the market is SPOT it will need to show both assets, condensed and maybe simplified side by side cards, Joelle is working on this now and should have something by the time someone gets to this
  • If the user clicks on the asset pane it should expand to all assets and their breakdownas shown in ASSETS
    • If the user clicks on deposit / transfer / withdraw button here te same behaviour as above should happen
    • If the user clicks "manage" it should send the user to the new full portfolio page with the asset mgmt panel set to the same asset breakdown as we use in the sidebar Asset breakdown in portfolio page & sidebar #6110
  • If the user clicks on the market info tab it should expand and show the full market info accordion inside the component, with the other cards shrinking to their minimised state as in MARKETS (poorly named sketch) ... the preferred design is the second one (3.2)
  • In each case there is no "minimise" button on the open pane... you minimise by clicking on something else
  • The size of each section should be predefined and locked, with the expanded one always getting the same max size
  • Scrolling should scroll WITHIN each section if required, the sidebar should not scroll as a whole since the parts should be locked in place as above
  • As a result of this change we can now remove the "Collateral" pane on the bottom table
  • Ignore the sketches related to SWAP for now ... these will be dealt with seperately

Sketch

All sketches now finalised in Figma: https://www.figma.com/file/ywcCZvXdvG28bg0AmCY9mF/Console-v3?type=design&node-id=6158-40256&mode=design&t=mj8twKYmzekRYkP3-0

Additional details / background info

@JonRay15
Copy link
Contributor Author

Note in the screenshots above there is a "Show Details" button on the asset ... but actually if the tile or the asset symbol was clickable we could remove the need for that

@JonRay15
Copy link
Contributor Author

Make sure this includes showing BOTH assets in spot markets in the trading sidebar.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch-develop Work for the next planned release of frontend apps
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants