Skip to content

Conversation

@luke-jr
Copy link
Member

@luke-jr luke-jr commented Jan 12, 2015

The tilde appears identical to a negative sign for me:

This changes it to something more obvious:

… (which may be mistaken for a negative sign)
@gmaxwell
Copy link
Contributor

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.

@Michagogo
Copy link
Contributor

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.

@gmaxwell
Copy link
Contributor

@Michagogo falling back? I don't think so... being visually indistinguishable to? probably, but thats not so severe a problem as the 'negative' values.

@jonasschnelli
Copy link
Contributor

OSX 10.10
Before:
bildschirmfoto 2015-01-13 um 08 57 07

After:
bildschirmfoto 2015-01-13 um 08 56 15

Win7:
bildschirmfoto 2015-01-13 um 09 01 10

Are there any OS without correct default font utf8 support with a mentionable amount of market share?

Tested ACK.

@jonasschnelli
Copy link
Contributor

Just another thought:

why not s/~/aprox.?

@Michagogo
Copy link
Contributor

May want to also find someone on Win8+ and an earlier OS X, since I'm
pretty sure 10.10 changed the system font.

On Tuesday, January 13, 2015, Jonas Schnelli [email protected]
wrote:

Just another thought:

why not s/~/aprox.?


Reply to this email directly or view it on GitHub
#5651 (comment).

@laanwj laanwj added the GUI label Jan 13, 2015
@laanwj
Copy link
Member

laanwj commented Jan 13, 2015

@zander \u is c++11 so we can't use it yet. The current formulation makes no assumption about the compiler, and just creates a UTF-8 string in bytes form. The end-result of \u in memory would be the same.

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):

#if QT_VERSION < 0x050000
    // Internal string conversion is all UTF-8
    QTextCodec::setCodecForTr(QTextCodec::codecForName("UTF-8"));
    QTextCodec::setCodecForCStrings(QTextCodec::codecForTr());
#endif

That said, QChar(0x2243) looks a bit more readable so I'd prefer that.

Tested ACK anyway.

@Diapolo
Copy link

Diapolo commented Jan 13, 2015

utACK

@laanwj
Copy link
Member

laanwj commented Jan 13, 2015

@Diapolo it'd help moving this forward if you test on windows

@Diapolo
Copy link

Diapolo commented Jan 13, 2015

@laanwj Done, looks good. Tested ACK! Btw. can you help me with pushing the payment request unit tests?

@Michagogo
Copy link
Contributor

Looks like on Windows 7, it does show properly (0.10.0rc3 with this cherry-picked on top):

image

I think I have a Windows 10 Preview VM, if so I'll try installing it there too.

@theuni
Copy link
Member

theuni commented Jan 13, 2015

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
Copy link
Contributor

Looks fine in zh_CN on Windows 7:
image

@theuni
Copy link
Member

theuni commented Jan 13, 2015

@Michagogo thanks for testing.

@Michagogo
Copy link
Contributor

Should still be checked on OS X and the various Linux environments, though.

@jonasschnelli
Copy link
Contributor

I did test on OSX10.10 and 10.9 (comparable to Win8 and Win7 in case of actuality):
See osx10.10 screenshot above and here also comes the 10.9 screenshot:

bildschirmfoto 2015-01-13 um 21 45 42

@theuni
Copy link
Member

theuni commented Jan 13, 2015

Verified looks ok in Ubuntu with qt4 and qt5. Kde/kwin.

@Michagogo
Copy link
Contributor

@theuni Uh, I thought Ubuntu is Unity?

On Tue, Jan 13, 2015 at 11:09 PM, Cory Fields [email protected]
wrote:

Verified looks ok in Ubuntu with qt4 and qt5. Kde/kwin.


Reply to this email directly or view it on GitHub
#5651 (comment).

@theuni
Copy link
Member

theuni commented Jan 13, 2015

Only by default.

@jonasschnelli
Copy link
Contributor

We should also test this on OSX over a gitian build.

@laanwj
Copy link
Member

laanwj commented Jan 16, 2015

@jonasschnelli Right - let's postpone this until the MacOSX unicode issues are fixed.

@jonasschnelli
Copy link
Contributor

@jonasschnelli
Copy link
Contributor

Tested over gitian on OSX and windows.

ACK

@laanwj laanwj merged commit 1d84aea into bitcoin:master Jan 20, 2015
laanwj added a commit that referenced this pull request Jan 20, 2015
1d84aea Coin Control: Use U+2248 "ALMOST EQUAL TO" rather than a simple tilde (which may be mistaken for a negative sign) (Luke Dashjr)
@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.

8 participants