Skip to content
This repository has been archived by the owner on Jun 17, 2020. It is now read-only.

Add translations for info icon in market based price input field #38

Merged

Conversation

ripcurlx
Copy link
Contributor

Copy link
Contributor

@ManfredKarrer ManfredKarrer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@ManfredKarrer ManfredKarrer merged commit d7a3916 into bisq-network:master Mar 28, 2018
offerbook.info.sellAboveMarketPrice=You will get {0} more than the current market price (updated every minute).
offerbook.info.buyBelowMarketPrice=You will pay {0} less than the current market price (updated every minute).
offerbook.info.buyAtFixedPrice=Your will buy at this fixed price.
offerbook.info.sellAtFixedPrice=You will sell at this fixed price.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ripcurlx, these translations look OK from a native speaker's point of view, but I'm not sure why there is a "sell at / buy at" vs "get / pay" difference. Why not do the following instead?

offerbook.info.sellBelowMarketPrice=You will sell at {0} less than the current market price (updated every minute).
offerbook.info.buyAboveMarketPrice=You will buy at {0} more than the current market price (updated every minute).

I am assuming that the {0} placeholder will be replaced with a percent value, btw. e.g. "You will sell at 2% less than the current market price..."

Copy link
Contributor Author

@ripcurlx ripcurlx Mar 29, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, the placeholder is replaced by the absolute percentage distance.
Regarding buy/sell vs. get/pay: I wasn't sure if it might be easier for the user to understand if I use get/pay in some cases instead of buy/sell e.g.

offerbook.info.sellAboveMarketPrice=You will get 2% more than the current market price (updated every minute).

instead of

offerbook.info.sellAboveMarketPrice=You will sell at 2% more than the current market price (updated every minute).

But it's just a personal feeling (and I'm no native speaker)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd favor being consistent. I don't think this is really about native-speaker nuance here, just the general principle that it's better to say the same thing one way versus two ways.

I might make the same point about using the phrase "above the current market price" vs. "more than the current market price", but I don't feel super strongly about it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm also a big fan of consistency, but based on some comments on different issues, I have the feeling that it always takes a lot of thinking to understand if above/below in combination with buy/sell is good or bad for me. I think I should start to do more user testing soon to get a better feeling on the biggest pain points within the upcoming release.

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

Successfully merging this pull request may close these issues.

3 participants