Simplify titles of tabs in main window #2428

r000n commented at 6:50 PM on March 29, 2013: contributor

Words in English language are more short than in other langs usually. Tabs don't fits in line in Spanish/German/Russian when they has two words. Wallet has limited functionality. It can send & receive coins. So we can safely rename "Send coins" to "Send" and "Receive coins" to "Receive". Address book is just user contacts.

This change is affects a translations. Ported from Freicoin

in src/qt/bitcoingui.cpp:None in 9940db0794 outdated

 187 | @@ -188,7 +188,7 @@ void BitcoinGUI::createActions()
 188 |      historyAction->setShortcut(QKeySequence(Qt::ALT + Qt::Key_4));
 189 |      tabGroup->addAction(historyAction);
 190 |  
 191 | -    addressBookAction = new QAction(QIcon(":/icons/address-book"), tr("&Address Book"), this);
 192 | +    addressBookAction = new QAction(QIcon(":/icons/address-book"), tr("&Contacts"), this);

Diapolo commented at 8:38 PM on March 29, 2013:

IMHO Contacts is not what the Address Book really IS, so NACK to this one, I'm fine with the others though.

burger2 commented at 10:29 PM on March 29, 2013:

I agree with Diapolo with Address Book it's a keeper!

laanwj commented at 6:11 AM on March 30, 2013:

Agree with Diapolo. "Coins" is indeed unnecessary and ACK on removing it.

"Contacts" doesn't really make sense to me for financial sw. What it really is, is a list of past (or stored for future usage) recipient addresses. So "address book" is a pretty nice analogy. Then again there may be wisdom in completely moving it to the background as it encourages re-use, see also @gmaxwell's point in #2429 but that's for future discussion.

BitcoinPullTester commented at 7:30 AM on March 30, 2013: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/9940db0794c9a6acdb76406569ba6668ff732dde for binaries and test log. This is an automated test script which runs test cases on each commit every time is updated. It, however, dies sometimes and fails to test properly, if you are waiting on a test, please check timestamps and if the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ and contact BlueMatt on freenode if something looks broken.

r000n commented at 9:16 AM on March 30, 2013: contributor

Probably, rename it to "Addresses"? If it's not a real address book :)

Words in English language are more short than in other langs usually.

Tabs don't fits in line in Spanish/German/Russian when they has two words.
Wallet has limited functionality. It can send & receive coins. So we can
safely rename "Send coins" to "Send" and "Receive coins" to "Receive".
Address book is just stored addresses.

0de26d5826

laanwj commented at 2:49 PM on April 1, 2013: member

ACK

Diapolo commented at 5:53 AM on April 2, 2013: none

ACK @laanwj If you get this in today, I'll create another translations update for this. Also, I remember I asked about a better general translations process in line with release cycles and whether we should drop languages, that are below a certain percentage translated, what do you think?

laanwj commented at 12:41 PM on April 2, 2013: member

@Diapolo yes I agree. Is 'updating translations' something that could be made part of the release process in doc/release-process.txt? I suppose so?

With regard to dropping translations, I don't know. I suppose below a certain % you could just as well have no translation. Do you know how other projects using transfex handle this, and if so what threshold they use?

laanwj referenced this in commit a977b8c475 on Apr 2, 2013

laanwj merged this on Apr 2, 2013

laanwj closed this on Apr 2, 2013

laudney referenced this in commit 25a543ec1e on Mar 19, 2014

DrahtBot locked this on Sep 8, 2021