Skip to content
This repository has been archived by the owner on Jan 25, 2021. It is now read-only.

Native Trade view title #93

Open
AGKDesigns opened this issue Oct 2, 2018 · 5 comments
Open

Native Trade view title #93

AGKDesigns opened this issue Oct 2, 2018 · 5 comments
Labels
bug Something isn't working

Comments

@AGKDesigns
Copy link

At the moment if you want to sell GAS and then swap the token to something else it still says Sell GAS.

We will need to change the heading of the native trade view away from buy and sell "OOO" I think, if we allow users to sway, it will need to swap the heading every time which is not good practice generally without changing screens.

Perhaps it could be something more generic like "Trade"

@AGKDesigns AGKDesigns added the bug Something isn't working label Oct 2, 2018
@apisit
Copy link

apisit commented Oct 2, 2018

are you on the latest version?

@AGKDesigns
Copy link
Author

AGKDesigns commented Oct 2, 2018

Ah it seems fixed now, I still get sort of confused by it, being able to change the pairs. Maybe thats just me though. I feel like the icons and the headers are disconnected some how.

@apisit
Copy link

apisit commented Oct 2, 2018

oh I see now. it says sell GAS but doesn't say "to"

@apisit
Copy link

apisit commented Oct 3, 2018

how about we say " Trade [LEFT SIDE] for [RIGHT SIDE]" ?
e.g. sell SWTH for NEO would be "Trade SWTH for NEO"
buy SWTH with NEO would be "Trade NEO for SWTH"

@AGKDesigns
Copy link
Author

yeah that makes more sense to me.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants