Mind that these messages also end up in the log. In principle the log is already UTF-8 (as are all strings and text in bitcoind). But, just noting, that it might make browsing the log a less pleasant experience on systems with misconfigured locale like some BSDs by default.
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
DrahtBot cross-referenced this on May 3, 2021 from issue Wallet passive startup by ryanofsky
DrahtBot
commented at 9:31 AM on May 3, 2021:
contributor
<!--4a62be1de6b64f3ed646cdc7932c8cf5-->
🕵️ @sipa has been requested to review this pull request as specified in the REVIEWERS file.
kiminuo
commented at 2:15 PM on May 3, 2021:
contributor
Concept ACK, it's much nicer!
Went line by line and it looks correct to me.
laanwj removed the label P2P on May 4, 2021
laanwj removed the label RPC/REST/ZMQ on May 4, 2021
laanwj removed the label Refactoring on May 4, 2021
laanwj removed the label Validation on May 4, 2021
laanwj removed the label Wallet on May 4, 2021
laanwj
commented at 2:54 PM on May 4, 2021:
member
ACKd66f283ac07edce432b964f7f814631f5a5bc33b
We do want to do an English translation update after this to prevent people from wasting time translating the old messages.
and it does not touch LogPrint and LogPrintf calls.
Still, InitMessages end up in the log. I don't see it as a blocking issue if no one else does though my comment was just a remark.
kristapsk
commented at 11:15 AM on May 5, 2021:
contributor
What are the benefits of this? Just looking nicer (which is discutable too)?
kiminuo
commented at 11:23 AM on May 5, 2021:
contributor
What are the benefits of this? Just looking nicer (which is discutable too)?
For many people there will be zero benefit. It depends on audience. Either you care about typography or you don't.
Anyway, when I open randomly a few applications on my computer, ellipses are used in dialogs and menus often so Bitcoin Core will look like other applications do which is a good thing I think.
hebasto
commented at 11:41 AM on May 5, 2021:
member
What are the benefits of this? Just looking nicer (which is discutable too)?
As stated in the pr description, this change was suggested by translators on Transifex.com (you could see open issues there).
I personally did not make any research on this topic. I assume that translators could be more experienced than me on it :)
laanwj
commented at 11:50 AM on May 5, 2021:
member
It just seems to be the way to do it nowadays (in the age of unicode). It's intended as ELLIPSIS, there's a dedicated character for it which is better than using three separate dots. This means the rendering engine has more meta-information about how to display it. It's definitely a very minor thing though and if it becomes controversial in the least we should close this.
Edit: I guess for accessibility it is also marginally important as a hint, instead of <dot> <dot> <dot> the text to speech can interpret it semantically.
laanwj merged this on May 10, 2021
laanwj closed this on May 10, 2021
laanwj referenced this in commit 99e98084fc on May 10, 2021
This is a metadata mirror of the GitHub repository
bitcoin/bitcoin.
This site is not affiliated with GitHub.
Content is generated from a GitHub metadata backup.
generated: 2026-05-20 06:54 UTC