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

Market Spread Buy/Sell BTC columns flipped? #1231

Closed
tr37ion opened this issue Jan 18, 2018 · 3 comments
Closed

Market Spread Buy/Sell BTC columns flipped? #1231

tr37ion opened this issue Jan 18, 2018 · 3 comments

Comments

@tr37ion
Copy link

tr37ion commented Jan 18, 2018

It looks like the Selll/Buy BTC column values are flipped. Eg. Siacoin it shows two Buy BTC orders:

image

While there are actually ...

Sell BTC orders:
image

Buy BTC orders:
image

@tr37ion tr37ion changed the title MArket Spread Buy/Sell BTC columns flipped? Market Spread Buy/Sell BTC columns flipped? Jan 18, 2018
@ripcurlx
Copy link
Contributor

Good catch! This seems to be a bug.

@ripcurlx
Copy link
Contributor

I just walked through the current behavior with @ManfredKarrer again. It is not a bug, it's just a mind bending task atm to see it correctly. In the Sell BTC tab the Sell means, you want to the sell BTC and buy SC. So these two users want to Buy BTC by selling SC. This is also the point of view presented in the spread table, where you find these two offers in the Buy BTC column. This is quite confusing which is true, why I'll start with the first mockup for a new trading process (see #1207) tomorrow. Still I'll close this issue, as it is not a bug.

@tr37ion
Copy link
Author

tr37ion commented Jan 25, 2018

Just an addition: Bisq should display lists and values from only one view - the user's view. Mixing two different views confuses the user IMHO.

Please consider only showing lists the current Bisq user expects/is interested in. It is not important to see what `..these two users want ...buy/sell..". Therefore, please keep in mind fixing this with #1207.

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

No branches or pull requests

2 participants