update some strings used as warning messages in sendcoinsdialog.cpp #1288

pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:sendcoinsdialog_strings changing 1 files +5 −5
  1. Diapolo commented at 1:41 PM on May 13, 2012: none

    No description provided.

  2. laanwj commented at 4:33 PM on May 13, 2012: member

    You're trying to break all the translations aren't you? :p

  3. update some strings used as warning messages in sendcoinsdialog.cpp 51ce20d5da
  4. in src/qt/sendcoinsdialog.cpp:None in 5cdfa0f380 outdated
     135 |              QMessageBox::Ok, QMessageBox::Ok);
     136 |          break;
     137 |      case WalletModel::AmountWithFeeExceedsBalance:
     138 |          QMessageBox::warning(this, tr("Send Coins"),
     139 | -            tr("Total exceeds your balance when the %1 transaction fee is included").
     140 | +            tr("Total exceeds your balance when the %1 transaction fee is included.").
    


    laanwj commented at 4:34 PM on May 13, 2012:

    Maybe make this "The total" too?


    Diapolo commented at 6:19 PM on May 13, 2012:

    Agreed!

  5. Diapolo commented at 6:22 PM on May 13, 2012: none

    @laanwj I think we should break translations before RC-phase, so yes ... I'm cleaning up ^^.

  6. laanwj commented at 6:27 PM on May 13, 2012: member

    I wish there was a way to change the English translation without invalidating all the translated messages... Or at least update the source string for all translations automatically, when there is no substantial difference.

  7. Diapolo commented at 6:31 PM on May 13, 2012: none

    This would be a nice thing, but Qts same text heuristic doesn't cover such things.

    Edit: All this leads to one thing, take more care if doing strings in the client that need a translations as this is a great plus in terms of UX!

  8. laanwj commented at 7:03 PM on May 13, 2012: member

    I think you could actually set the strings as "translation" strings in bitcoin-en.ts instead?

  9. Diapolo commented at 7:21 PM on May 13, 2012: none

    Sorry, I don't get it ... can you explain?

  10. laanwj commented at 7:22 PM on May 13, 2012: member

    If you open bitcoin-en.ts in Qt translator and set the new messages there, it will change the message shown in English clients without changing the message in the source code, so without affecting the other translations.

  11. Diapolo commented at 7:30 PM on May 13, 2012: none

    This would be less invasive, but produce an ugly inconsistency for the en translation-file and the source ... I don't really like that idea. A form of %RPC_Con_Information_Label_text% in the source and translations only in the lang-files, but that would even further raise the complexity.

  12. laanwj commented at 7:37 PM on May 13, 2012: member

    I don't like it either. But I also don't like it if 20 translators have to re-translate a message just because a space was removed, or a dot was added, for example.

    Transifex also has no way to edit a source message. However, they claim they have "Fussy re-using of existing strings in source language files" with a "similarity threshold" (see http://help.transifex.com/server/releases/1.1.html), so let's hope that helps...

  13. laanwj referenced this in commit a3957d089a on May 13, 2012
  14. laanwj merged this on May 13, 2012
  15. laanwj closed this on May 13, 2012

  16. coblee referenced this in commit f901d5bf94 on Jul 17, 2012
  17. suprnurd referenced this in commit 60409df822 on Dec 5, 2017
  18. DrahtBot locked this on Sep 8, 2021
Contributors

github-metadata-mirror

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-04-21 18:16 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me