Simplify titles of tabs in main window #2428

Merged
merged 1 commit into from Apr 2, 2013

4 participants

@r000n

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

@Diapolo Diapolo and 2 others commented on an outdated diff Mar 29, 2013
src/qt/bitcoingui.cpp
@@ -188,7 +188,7 @@ void BitcoinGUI::createActions()
historyAction->setShortcut(QKeySequence(Qt::ALT + Qt::Key_4));
tabGroup->addAction(historyAction);
- addressBookAction = new QAction(QIcon(":/icons/address-book"), tr("&Address Book"), this);
+ addressBookAction = new QAction(QIcon(":/icons/address-book"), tr("&Contacts"), this);
@Diapolo
Diapolo added a note Mar 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
burger2 added a note Mar 29, 2013

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

@laanwj
Bitcoin member
laanwj added a note Mar 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@BitcoinPullTester

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

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

@r000n r000n 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.
0de26d5
@laanwj
Bitcoin member

ACK

@Diapolo

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
Bitcoin 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 laanwj merged commit a977b8c into bitcoin:master Apr 2, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment