-
Notifications
You must be signed in to change notification settings - Fork 210
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
Implement new dex view design #1561
Conversation
Change "Trading Information" text style
Change tab indicator color in trading informations section
Change padding in trading informations section
Change orders/history style
Change order book style
Change best order list style
Change place order form subtitle
Change order book view title style
Edit separator maximum width between header and list
Remove trailing whitespaces between date and hour
Allow custom colors
There is no chart at the moment Figma:
@SylEze what is the min screen size you need design for? |
@SylEze what is the min screen size you need design for? |
Undefined reference to order_form from orderbook
…_line_warning_dancing_on_hover Pro Trading View: Move orderbook warning sign from line to tooltip
Change order/history row style
Ticker selector content text color was wrong
Change orders/history tab paginator combobox color
looks good so far on my side, great job! @Canialon could you please carefully test DEX -> Pro view? the only pb I noticed so far is a small glitch on modal: |
chart-expand-retract.mp4
|
|
I will not fix every issue mentionned in this PR, only the issues I put in the description It has already sufficient changes: those ones will be fixed in another pull requests |
@SylEze @tonymorony suggest we merge this in current state and then address the following issues separately (or move to new issues):
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving, outstanding issues to be dealt with separately. @SylEze please remove incomplete issue numbers # from your comment before merging so they don't close prematurely
Closes #1268
Closes #1342
Closes #1488