Language selector should use language names instead of codes #1509

issue luke-jr opened this issue on June 23, 2012
  1. laanwj commented at 2:10 PM on June 25, 2012: member

    Use QLocale::countryToString / QLocale::languageToString (http://doc.qt.nokia.com/4.7-snapshot/qlocale.html#languageToString), it returns the full name for a country / language.

  2. Diapolo commented at 6:07 PM on June 25, 2012: none

    Tried it and should be rather easy to do ... perhaps I'll create a patch later.

  3. Diapolo commented at 11:04 AM on June 27, 2012: none

    Can be closed now!

  4. luke-jr commented at 4:33 PM on June 27, 2012: member

    IMO, this should be using nativeLanguageName

  5. Diapolo commented at 4:48 PM on June 27, 2012: none

    Nice idea, but This function was introduced in Qt 4.8. and we are still compiling using an older 4.7.4 (4.8.2 is current).

    Edit: We could aswell use nativeCountryName() then, which would be even better.

    Edit 2: That's how it would look then: language selection

  6. Diapolo commented at 1:49 PM on July 12, 2012: none

    @luke-jr Can you close this for now? @laanwj Any date / idea when or if we can switch to a newer Qt version?

  7. laanwj commented at 2:11 PM on July 12, 2012: member

    We won't require a higher Qt version for the forseeable future, however you can use 4.8 features in a #if QT_VERSION >= 0x040800 guard so that people that do compile with 4.8 get the better functionality.

  8. Diapolo commented at 2:20 PM on July 12, 2012: none

    Good point, I'll consider this, but it would get pretty ugly, when there are many places in the code with such guards.

  9. laanwj commented at 2:25 PM on July 12, 2012: member

    I wouldn't consider that much of a problem in this case. It's simply how backward compatible programming works. It's much worse in web development, at least you don't have to support IE 6 :-)

  10. Diapolo commented at 2:30 PM on July 12, 2012: none

    AFAIK we support / require IE 5.01 :-P!? #define _WIN32_IE 0x0501

  11. laanwj commented at 6:20 AM on July 17, 2012: member

    Fixed with #1591, closing issue

  12. laanwj closed this on Jul 17, 2012

  13. lateminer referenced this in commit c891c1478f on Jan 22, 2019
  14. lateminer referenced this in commit 13c4302087 on May 6, 2020
  15. MarcoFalke locked this on Sep 8, 2021

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-14 15:16 UTC

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