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

Make sell/buy BTC screen more self-explanatory #1203

Closed
ripcurlx opened this issue Jan 16, 2018 · 63 comments
Closed

Make sell/buy BTC screen more self-explanatory #1203

ripcurlx opened this issue Jan 16, 2018 · 63 comments
Assignees

Comments

@ripcurlx
Copy link
Contributor

Add information inline for price distance and volume range and add hint line before the create offer call-to-action button. Here's a rough sketch how it could look like:
sell bitcoin

@ripcurlx ripcurlx changed the title Improvement: Make sell/buy BTC screen more self-explanatory Make sell/buy BTC screen more self-explanatory Jan 18, 2018
@tr37ion
Copy link

tr37ion commented Jan 18, 2018

Please don't forget to dot-align the figures #1028

@ripcurlx
Copy link
Contributor Author

Good point! I'll have a look how to do this in a consistent way. Sometimes it is a little bit tricky when we also have to display ranges.

@tr37ion
Copy link

tr37ion commented Jan 19, 2018

For now you could just switch to Fira Mono font, which looks like a sans-serif font and then just fill up the values with zeros to the right, so that values align nicely to the right. It is important to have a mono-space font, because then values are vertically comparable.

Sure a dot-aligned solution would be the best solution in the end.

@ripcurlx
Copy link
Contributor Author

@fira mono: I'll have a look how it looks like mixing these two fonts within the app and the size of the font, as we have to bundle it within the app.

@tr37ion
Copy link

tr37ion commented Jan 22, 2018

Fira Mono is one of the few fonts which scales very well. It is perfectly readable even with very small sizes.

image

@ripcurlx
Copy link
Contributor Author

ripcurlx commented Feb 6, 2018

An updated version of the buy/sell screen up for discussion. @ManfredKarrer & @tr37ion let the discussion begin 😉
sell screen

@ripcurlx
Copy link
Contributor Author

ripcurlx commented Feb 6, 2018

Or the same with the call-to-action button below.
sell screen

@tr37ion
Copy link

tr37ion commented Feb 6, 2018

I got this layout. Positioning the filters and the Create button is not easy.

image

@ripcurlx
Copy link
Contributor Author

If we would align it like this, I would start from the left, as this is the order how to approach this screen. Adapt the filter if necessary -> Take offer or move down to Create new offer. If we add the missing decimal places I think it would be maybe better to add them with a different color, not to overwhelm the user with zeros that don't add additional value. Regarding the flags I'm not such a great fan, as I think that users recognize/know the country codes more than the actual flags.
sell screen no flags

@tr37ion
Copy link

tr37ion commented Feb 12, 2018

Looks fine. The different colour for the zeros is a good idea. Maybe move the percentages to a new column.

What about making the button like in my mockup and moving the NO. of offers .. to the status bar?

@ripcurlx
Copy link
Contributor Author

Moving the percentage into a new column might solve the need of an additional + for the positive price distance, so prices are still aligned properly. To be able to squeeze it into the view I also had to rename the price column and added an additional question icon for explaining Distance. I've also added the idea of the issue #1214 showing which distance is good or bad for the user. Having the No. of offers in the footer, is too far off and will be overlooked for sure. What do you think about following mockup?
sell screen no flags

@ripcurlx
Copy link
Contributor Author

I've also renamed the column title Buyer to Trader to reduce the usage of Buy within the Sell bitcoin screen.

@ripcurlx
Copy link
Contributor Author

Or we make it super verbose and explain everything 😉
sell screen no flags

@ripcurlx
Copy link
Contributor Author

@ManfredKarrer What do you think about going in this direction within the Sell/Buy bitcoin screen?

@tr37ion
Copy link

tr37ion commented Feb 13, 2018

Some ideas:

  • Add filter for only showing offers one can take (registered accounts)
  • Move current vie to Expert view and show only Price, Distance, Payment Method and Trader
  • Add filter for showing only offers the user can take (> min limits)

@ripcurlx
Copy link
Contributor Author

I think the amount of BTC to sell should also be visible for each offer, even in the non-expert view, as it is also important for your decision to sell (or buy).

@ripcurlx
Copy link
Contributor Author

Maybe we can have a condensed view for new users and within filters the option to have an enhanced table where all values get their own column.
sell screen

@ripcurlx
Copy link
Contributor Author

That way, we can still implement sorting that takes the min & max values into account (as it is now) and reduce the number of columns a user has to overview from the beginning. For power users we have the filter and an option to split up the BTC and EUR range into a separate column.

@ripcurlx
Copy link
Contributor Author

Or we name it instead of "show filters" --> "advanced settings"

@ManfredKarrer
Copy link
Contributor

Problem with "Add filter for only showing offers one can take (registered accounts)" is that new users would not see anything before they have registered an account.

@ManfredKarrer
Copy link
Contributor

Re "Move current vie to Expert view and show only Price, Distance, Payment Method and Trader" - that is more or less the view at the market overview screen.

@tr37ion
Copy link

tr37ion commented Feb 13, 2018

Or list the max|min available values for buy|sell orders. So that with buy orders only the max available value is shown. If you don't have the min asked amount, the offer will not be listed eg.

@ManfredKarrer
Copy link
Contributor

The idea with the bar in the background of the price distance is nice but has the problem that negative (green) are an exception. So all will be red (rosa) which gives negative feeling.
The info icons at the tables are too much for me. Then nearly every item gets an info. Instead we could maybe use tooltips as kind of lightweight info? Or maybe have an introduction / guided tour mode where all items are explained. But better to get to wording which does not require explanations (hard :-)).

@sqrrm
Copy link
Member

sqrrm commented Feb 13, 2018

The background color bar could use the average distance as the base for colorization, such that 1% is still green if that's better than the average distance in the market.

@tr37ion
Copy link

tr37ion commented Feb 13, 2018

@ManfredKarrer Well, isn't it the purpose of those colours to guide the user a bit? I think we should not think too much of what the user might 'feel' of red/green colouring. If it is mostly red then it be so. It is just a quick way to find the most attractive offer. Plus get a feeling for How much the offer is off the current market price.

@sqrrm has a nice idea, while I'm not sure if all rainbow colours have enough contrast to be shown.

A compromise might me using a grey color.

@ripcurlx
Copy link
Contributor Author

I'll let the idea of indicating the price distance visually rest for a while in the case that we might come up with an even better idea soon. Still I will start implementing following version of the Buy/Sell screen first. I've tried out a new wording for "price distance" as well --> price offset. Do you think this is easier to understand than the term we used before? sell screen no flags

@ripcurlx
Copy link
Contributor Author

As I'm about to submit the next pull request I'd love some feedback from a native speaker (@cbeams ?) on the tooltips used. Thanks!
Positive percentage
bildschirmfoto 2018-03-28 um 14 55 04
0 percent
bildschirmfoto 2018-03-28 um 14 55 01
Negative percentage
bildschirmfoto 2018-03-28 um 14 54 57
Fixed price
bildschirmfoto 2018-03-28 um 14 54 55

@stale
Copy link

stale bot commented Jan 17, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

1 similar comment
@stale
Copy link

stale bot commented Apr 18, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the was:dropped label Apr 18, 2019
@stale
Copy link

stale bot commented Apr 25, 2019

This issue has been automatically closed because of inactivity. Feel free to reopen it if you think it is still relevant.

@stale stale bot closed this as completed Apr 25, 2019
@ripcurlx
Copy link
Contributor Author

Still relevant.

@ripcurlx ripcurlx reopened this Apr 25, 2019
@stale stale bot removed the was:dropped label Apr 25, 2019
@stale
Copy link

stale bot commented Jul 24, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the was:dropped label Jul 24, 2019
@stale
Copy link

stale bot commented Jul 31, 2019

This issue has been automatically closed because of inactivity. Feel free to reopen it if you think it is still relevant.

@stale stale bot closed this as completed Jul 31, 2019
@ripcurlx
Copy link
Contributor Author

Still relevant.

@ripcurlx ripcurlx reopened this Aug 20, 2019
@stale stale bot removed the was:dropped label Aug 20, 2019
@stale
Copy link

stale bot commented Nov 18, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the was:dropped label Nov 18, 2019
@ripcurlx
Copy link
Contributor Author

Still relevant.

@stale stale bot removed the was:dropped label Nov 18, 2019
@stale
Copy link

stale bot commented Feb 16, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale
Copy link

stale bot commented May 17, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the was:dropped label May 17, 2020
@stale
Copy link

stale bot commented May 24, 2020

This issue has been automatically closed because of inactivity. Feel free to reopen it if you think it is still relevant.

@stale stale bot closed this as completed May 24, 2020
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

5 participants