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

[qt] change µBTC to bits #12035

Merged
merged 2 commits into from Jan 4, 2018
Merged

[qt] change µBTC to bits #12035

merged 2 commits into from Jan 4, 2018

Conversation

jb55
Copy link
Contributor

@jb55 jb55 commented Dec 27, 2017

Now that we have bip176, change "µBTC" to the more colloquial "bits"

@Sjors
Copy link
Member

Sjors commented Dec 27, 2017

I would suggest bits (µBTC), Bits (µBTC) or µBTC (bits), so more SI oriented folks don't need to Google "bits". Can you add some screenshots to see what this would look like in the UI?

@Sjors
Copy link
Member

Sjors commented Dec 27, 2017

Also paging @jimmysong who opened this can of worms :-)

@jimmysong
Copy link
Contributor

I would love if we can switch to bits, but not required. I understand if people don't want to change it.

@jb55
Copy link
Contributor Author

jb55 commented Dec 27, 2017 via email

@fanquake fanquake added the GUI label Dec 27, 2017
@Sjors
Copy link
Member

Sjors commented Dec 28, 2017

@jb55 you can actually drag & drop images onto Github, so they're inline.

There's more places where "bits" would appear, e.g. in the settings and as a dropdown in the send screen. If we go for bits (µBTC) that would probably look fine in a dropdown menu, but look terrible on the transactions screen. Whereas bits probably looks fine everywhere.

I remember the first time I saw the term "bits" in a wallet (Xapo). I found it very confusing, because I didn't know how many decimals it represents. I'm fine with switching to bits, but shouldn't assume all users are familiar with it. So it requires some education.

The easiest way to achieve a minimum level of education is to have two strings. Use the short version bits everywhere expect the dropdown menus, where you use the long string bits (µBTC). That would add a few lines of code to this PR, but nothing unsurmountable. Let me know if you need any pointers.

The tooltips are useful as well, but I don't think those appear over the dropdown menus.

@jb55
Copy link
Contributor Author

jb55 commented Dec 28, 2017

I've implemented your suggestion in af7f3c1ccafefd92603a5138a136a52f0c249fe9

Here's what it looks like:

dec28-061640

dec28-061628

dec28-061139

dec28-071931

@Sjors
Copy link
Member

Sjors commented Dec 28, 2017

ACK af7f3c1c, but please squash.

* Now that we have bip176, change "µBTC" to the more colloquial "bits"

* We retain the `µBTC (bits)` description in dropdowns and status bars.
  The more concise "bits" is used when appended to numbers.

Signed-off-by: William Casarin <jb55@jb55.com>
@jb55
Copy link
Contributor Author

jb55 commented Dec 28, 2017

Squashed in 275b2ee

@Sjors
Copy link
Member

Sjors commented Dec 28, 2017

Thanks, reACK 275b2ee.

@laanwj
Copy link
Member

laanwj commented Dec 30, 2017

No opinion on this but at least you should update doc/bips.md when you do this.

For the implementation in bitcoin#12035.
Note that this only applies to the QT GUI at this time.

Signed-off-by: William Casarin <jb55@jb55.com>
@jb55
Copy link
Contributor Author

jb55 commented Dec 30, 2017

@laanwj done in ebcee1d

@laanwj
Copy link
Member

laanwj commented Jan 3, 2018

Tested ACK ebcee1d

@jonasschnelli
Copy link
Contributor

utACK ebcee1d

@jonasschnelli jonasschnelli merged commit ebcee1d into bitcoin:master Jan 4, 2018
jonasschnelli added a commit that referenced this pull request Jan 4, 2018
ebcee1d bips: add bip176 (Bits Denomination) (William Casarin)
275b2ee [qt] change µBTC to bits (William Casarin)

Pull request description:

  Now that we have bip176, change "µBTC" to the more colloquial "bits"

Tree-SHA512: eba5e5f89c392728a4f0a3bd81a9779a117b8d72a490390fd031d4e7cc56c2bfee0016aba7ef9535903e8cf2262ce46497283424e378906d0e3bf5b0d2d981c7
hkjn pushed a commit to hkjn/bitcoin that referenced this pull request Jan 13, 2018
For the implementation in bitcoin#12035.
Note that this only applies to the QT GUI at this time.

Signed-off-by: William Casarin <jb55@jb55.com>
HashUnlimited pushed a commit to chaincoin/chaincoin that referenced this pull request Mar 15, 2018
For the implementation in bitcoin#12035.
Note that this only applies to the QT GUI at this time.

Signed-off-by: William Casarin <jb55@jb55.com>
deadalnix pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Feb 18, 2020
Summary:
ebcee1d bips: add bip176 (Bits Denomination) (William Casarin)
275b2ee [qt] change µBTC to bits (William Casarin)

Pull request description:

  Now that we have bip176, change "µBTC" to the more colloquial "bits"

Tree-SHA512: eba5e5f89c392728a4f0a3bd81a9779a117b8d72a490390fd031d4e7cc56c2bfee0016aba7ef9535903e8cf2262ce46497283424e378906d0e3bf5b0d2d981c7

Backport of Core [[bitcoin/bitcoin#12035 | PR12035]]

The naming changes were excluded.

Test Plan:
  ninja
  ./bitcoin-qt
`Options` -> `Display` -> `Units` -> `µBTC`
Verify home screen displays proper units.
{F4182074}

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, jasonbcox

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, jasonbcox

Subscribers: jasonbcox, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D5296
ftrader pushed a commit to bitcoin-cash-node/bitcoin-cash-node that referenced this pull request Apr 16, 2020
Summary:
ebcee1d bips: add bip176 (Bits Denomination) (William Casarin)
275b2ee [qt] change µBTC to bits (William Casarin)

Pull request description:

  Now that we have bip176, change "µBTC" to the more colloquial "bits"

Tree-SHA512: eba5e5f89c392728a4f0a3bd81a9779a117b8d72a490390fd031d4e7cc56c2bfee0016aba7ef9535903e8cf2262ce46497283424e378906d0e3bf5b0d2d981c7

Backport of Core [[bitcoin/bitcoin#12035 | PR12035]]

The naming changes were excluded.

Test Plan:
  ninja
  ./bitcoin-qt
`Options` -> `Display` -> `Units` -> `µBTC`
Verify home screen displays proper units.
{F4182074}

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, jasonbcox

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, jasonbcox

Subscribers: jasonbcox, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D5296

ABC cherry-picker: Nico Guiton <nico@bitframe.org>
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants