-
Notifications
You must be signed in to change notification settings - Fork 38.7k
Coin Control: Use U+2248 "ALMOST EQUAL TO" rather than a simple tilde #5651
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
Conversation
… (which may be mistaken for a negative sign)
|
I like this, in that even if it fails to display correctly you'll just get a box or a missing symbol or and equals sign; and all are much better failure modes than something that looks like a minus sign. But I am not a GUI wizard. |
|
Is falling back to = something that might happen? Also, we need to check the various platforms' default fonts (OS X 10.10, OS X <10.10, Win7, Win8, maybe Unity default, possibly others?) to see if this looks right. |
|
@Michagogo falling back? I don't think so... being visually indistinguishable to? probably, but thats not so severe a problem as the 'negative' values. |
|
Just another thought: why not |
|
May want to also find someone on Win8+ and an earlier OS X, since I'm On Tuesday, January 13, 2015, Jonas Schnelli [email protected]
|
|
@zander As for the client we explicitly force all string converstions to be UTF-8 at the start of the program (which is the default for Qt5 higher): That said, QChar(0x2243) looks a bit more readable so I'd prefer that. Tested ACK anyway. |
|
utACK |
|
@Diapolo it'd help moving this forward if you test on windows |
|
@laanwj Done, looks good. Tested ACK! Btw. can you help me with pushing the payment request unit tests? |
|
Someone mind checking this in some of the more difficult languages? I believe traditional chinese is one that usually ends up causing us trouble with exotic chars like these. |
|
@Michagogo thanks for testing. |
|
Should still be checked on OS X and the various Linux environments, though. |
|
Verified looks ok in Ubuntu with qt4 and qt5. Kde/kwin. |
|
@theuni Uh, I thought Ubuntu is Unity? On Tue, Jan 13, 2015 at 11:09 PM, Cory Fields [email protected]
|
|
Only by default. |
|
We should also test this on OSX over a gitian build. |
|
@jonasschnelli Right - let's postpone this until the MacOSX unicode issues are fixed. |
|
Prebuilt (gitian) binaries for testing are here: https://bitcoin.jonasschnelli.ch/pulls/5651/ |
|
Tested over gitian on OSX and windows. ACK |
1d84aea Coin Control: Use U+2248 "ALMOST EQUAL TO" rather than a simple tilde (which may be mistaken for a negative sign) (Luke Dashjr)






The tilde appears identical to a negative sign for me:

This changes it to something more obvious:
