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

General UI Criticism #154

Open
dafyddd opened this issue Feb 18, 2022 · 1 comment
Open

General UI Criticism #154

dafyddd opened this issue Feb 18, 2022 · 1 comment
Assignees

Comments

@dafyddd
Copy link
Contributor

dafyddd commented Feb 18, 2022

  1. I think the switch icon with the double arrows is not very intuitive and fast. For one, the spot markets are first but they're not necessarily the most popular markets (with the exception of one person market making the SOLUSDC). I think it would be cleaner if it was sorted by 24 hour perp volume and the spot markets placed below the perp markets with the same ordering (I understand we don't have an easy way to query spot volumes). I think the switch icon itself is not an inuitive symbol. I didn't know that I could use it to switch between markets. Maybe a dropdown symbol would be more intuitive? Maybe if the market name (e..g SOL-PERP) itself can be clicked to bring about the drop down? I don't know.

  2. The yellow connect wallet in the top right is way too bright when using the Mango theme. It's more acceptalbe in dark theme. Can we tone it down a bit?

@saml33
Copy link
Contributor

saml33 commented Feb 19, 2022

See Tyler has already implemented most of 1. Will have a go at the sorting.

That is the primary color for Mango theme. I kind of liked it bright but am happy to change it. Was thinking buttons should have their own color in the theme currently they use a background color. Might experiment with some unique button colors

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants