Minor GUI cosmetics, should fix #1063. #1088

pull ghost wants to merge 1 commits into bitcoin:master from changing 1 files +5 −4
  1. ghost commented at 4:56 AM on April 13, 2012: none

    Line breaks for long tool tips, a forgotten keyboard shortcut and a tool tip added to src/qt/optionsdialog.cpp

  2. Minor GUI cosmetics: line breaks for long tool tips, a forgotten keyboard shortcut and a tool tip added. Should fix #1063. c11af2d21b
  3. laanwj commented at 5:19 AM on April 13, 2012: member

    According to @diapolo transifex has problems with \n in messages, they tend to get lost in translations.

    Also, as I said before, I think it's crazy that we (and translators) have to estimate the width at which the text gets rendered, the width of the users screen, and manually break the text. This is not 1980.

    Are you sure Qt has no functionality to automatically wordwrap tooltips?

  4. ghost commented at 6:52 AM on April 13, 2012: none

    According to @Diapolo transifex has problems with \n in messages, they tend to get lost in translations.

    I had remarked this with the content of the About Bitcoin-Qt window but it was apparently just a rendering issue of translated strings on Transifex's side. The file was formatted correctly in Qt Linguist and in Bitcoin, too. Besides they seem to have fixed that glitch now, but I'm not sure so don't take it as granted.

    Are you sure Qt has no functionality to automatically wordwrap tooltips?

    I'm just a translator, I don't know much about coding but apparently if Qt considers the content of the tooltip as plain text it won't care about its length. However, if it thinks it deals with rich text it will wrap the content. Adding a < br> tag or < font color=black>lorem ipsum< /font> does the trick although it's not very elegant.

    So you may remove the 3 "\n" from the patch until you decide how you want to manage this tooltip issue.

  5. laanwj referenced this in commit 6b8e7eefcc on Apr 13, 2012
  6. laanwj commented at 7:20 AM on April 13, 2012: member

    Thanks for looking that up. We will handle the wordwrap issue in the code instead of the individual translations, by making sure all the tooltips are detected as rich text.

    I've applied the other part of this pull.

  7. laanwj closed this on Apr 13, 2012

  8. luke-jr referenced this in commit 1f91797535 on Apr 19, 2012
  9. coblee referenced this in commit 0573eeb69e on Jul 17, 2012
  10. suprnurd referenced this in commit ed0a0d2393 on Dec 5, 2017
  11. 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-17 15:16 UTC

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