-
Notifications
You must be signed in to change notification settings - Fork 37.7k
translations: Don't translate markup or force English grammar #6962
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@@ -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") |
There was a problem hiding this comment.
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
utACK |
1 similar comment
utACK |
@@ -499,6 +499,7 @@ std::string HelpMessage(HelpMessageMode mode) | |||
|
|||
std::string LicenseInfo() | |||
{ | |||
// todo: remove url's from translations on next change |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: URLs (no apostrophe)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@laanwj ✅ fixed nit
941ce42
to
6dd3a44
Compare
6dd3a44 translations: Don't translate markdown or force English grammar (MarcoFalke)
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.
No description provided.