Skip to content

Fix qtbase translations not being loaded (regression)#1611

Merged
phoerious merged 2 commits intorelease/2.3.1from
hotfix/qtbase-translations
Mar 4, 2018
Merged

Fix qtbase translations not being loaded (regression)#1611
phoerious merged 2 commits intorelease/2.3.1from
hotfix/qtbase-translations

Conversation

@phoerious
Copy link
Copy Markdown
Member

@phoerious phoerious commented Mar 3, 2018

Description

Despite a previous fix, qtbase translations were not loaded in 2.3.0. This patch fixes this issue.
Also fixes cipher/KDF translations not being applied.

Motivation and context

Qt standard widgets were not translated.

How has this been tested?

Manually, by selecting a language other than English.

Types of changes

  • ✅ Bug fix (non-breaking change which fixes an issue)

Checklist:

  • ✅ I have read the CONTRIBUTING document. [REQUIRED]
  • ✅ My code follows the code style of this project. [REQUIRED]
  • ✅ All new and existing tests passed. [REQUIRED]
  • ✅ I have compiled and verified my code with -DWITH_ASAN=ON. [REQUIRED]

@phoerious phoerious added this to the 2.3.1 milestone Mar 3, 2018
@phoerious phoerious requested a review from a team March 3, 2018 12:11
@phoerious phoerious force-pushed the hotfix/qtbase-translations branch from 930aa6f to 184b3c6 Compare March 4, 2018 12:15
@phoerious phoerious force-pushed the hotfix/qtbase-translations branch from 184b3c6 to 092375b Compare March 4, 2018 18:00
@phoerious
Copy link
Copy Markdown
Member Author

I added another i18n fix. The Argon2 selection source string was untranslatable, because it contained Unicode characters.

@phoerious phoerious force-pushed the hotfix/qtbase-translations branch from d371d95 to 8e94f46 Compare March 4, 2018 19:34
@phoerious phoerious force-pushed the hotfix/qtbase-translations branch from 8e94f46 to 0c46494 Compare March 4, 2018 19:34
@phoerious
Copy link
Copy Markdown
Member Author

phoerious commented Mar 4, 2018

Alright, another update. It was of course not just the Unicode character, but predominantly us dummies trying to get translated text from a static context. I fixed it, it works now. With proper UTF-8 decoding even with non-ASCII characters.

@phoerious phoerious merged commit 02b923b into release/2.3.1 Mar 4, 2018
@phoerious phoerious deleted the hotfix/qtbase-translations branch March 4, 2018 20:01
phoerious added a commit that referenced this pull request Mar 6, 2018
- Fix unnecessary automatic upgrade to KDBX 4.0 and prevent challenge-response key being stripped [#1568]
- Abort saving and show an error message when challenge-response fails [#1659]
- Support inner stream protection on all string attributes [#1646]
- Fix favicon downloads not finishing on some websites [#1657]
- Fix freeze due to invalid STDIN data [#1628]
- Correct issue with encrypted RSA SSH keys [#1587]
- Fix crash on macOS due to QTBUG-54832 [#1607]
- Show error message if ssh-agent communication fails [#1614]
- Fix --pw-stdin and filename parameters being ignored [#1608]
- Fix Auto-Type syntax check not allowing spaces and special characters [#1626]
- Fix reference placeholders in combination with Auto-Type [#1649]
- Fix qtbase translations not being loaded [#1611]
- Fix startup crash on Windows due to missing SVG libraries [#1662]
- Correct database tab order regression [#1610]
- Fix GCC 8 compilation error [#1612]
- Fix copying of advanced attributes on KDE [#1640]
- Fix member initialization of CategoryListWidgetDelegate [#1613]
- Fix inconsistent toolbar icon sizes and provide higher-quality icons [#1616]
- Improve preview panel geometry [#1609]
@phoerious phoerious added pr: bugfix Pull request fixes a bug and removed bug labels Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

i18n pr: bugfix Pull request fixes a bug regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants