Skip to content

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Nov 6, 2015

No description provided.

@@ -312,8 +312,9 @@ void SendCoinsDialog::on_sendButton_clicked()
if(u != model->getOptionsModel()->getDisplayUnit())
alternativeUnits.append(BitcoinUnits::formatHtmlWithUnit(u, totalAmount));
}
questionString.append(tr("Total Amount %1<span style='font-size:10pt;font-weight:normal;'><br />(=%2)</span>")
.arg(BitcoinUnits::formatHtmlWithUnit(model->getOptionsModel()->getDisplayUnit(), totalAmount))
questionString.append(tr("Total Amount %1")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this translation message was really ugly

@laanwj
Copy link
Member

laanwj commented Nov 6, 2015

utACK

1 similar comment
@fanquake
Copy link
Member

fanquake commented Nov 7, 2015

utACK

@@ -499,6 +499,7 @@ std::string HelpMessage(HelpMessageMode mode)

std::string LicenseInfo()
{
// todo: remove url's from translations on next change
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: URLs (no apostrophe)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@laanwj ✅ fixed nit

@laanwj laanwj added the Docs label Nov 9, 2015
@maflcko maflcko force-pushed the MarcoFalke-2015-translations branch from 941ce42 to 6dd3a44 Compare November 9, 2015 10:41
@laanwj laanwj merged commit 6dd3a44 into bitcoin:master Nov 9, 2015
laanwj added a commit that referenced this pull request Nov 9, 2015
6dd3a44 translations: Don't translate markdown or force English grammar (MarcoFalke)
@maflcko maflcko deleted the MarcoFalke-2015-translations branch November 9, 2015 11:25
zkbot added a commit to zcash/zcash that referenced this pull request Dec 18, 2019
Bitcoin 0.12 cleanup PRs 2

Cherry-picked from the following upstream PRs:

- bitcoin/bitcoin#6631
- bitcoin/bitcoin#6664
  - Only the first commit (we already had the second through bitcoin/bitcoin#6825).
- bitcoin/bitcoin#6669
- bitcoin/bitcoin#6887
  - Only the non-QT parts.
- bitcoin/bitcoin#6962
- bitcoin/bitcoin#6822
  - Only first and third commits (we already had the second through an earlier PR).
- bitcoin/bitcoin#7136
  - Excludes Travis CI changes, and fixes to documents we don't have anymore.
- bitcoin/bitcoin#7084
- bitcoin/bitcoin#7509
- bitcoin/bitcoin#7617
- bitcoin/bitcoin#7726

Part of #2074.
@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.

3 participants