Fix translations on Keyboard Shortcuts Widget

* Fixes #10335
keepassxreboot · Oct 7, 2024 · 7498fe2 · 7498fe2
1 parent 0941ff4
commit 7498fe2
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 30 deletions.
diff --git a/share/translations/keepassxc_en.ts b/share/translations/keepassxc_en.ts
@@ -8978,6 +8978,26 @@ This option is deprecated, use --set-key-file instead.</source>
         <source>Only PBKDF and Argon2 are supported, cannot decrypt json file</source>
         <translation type="unfinished"></translation>
     </message>
+    <message>
+        <source>Reset Shortcuts</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Double click an action to change its shortcut</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Filter...</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Shortcut Conflict</source>
+        <translation type="unfinished"></translation>
+    </message>
+    <message>
+        <source>Shortcut %1 conflicts with &apos;%2&apos;. Overwrite shortcut?</source>
+        <translation type="unfinished"></translation>
+    </message>
     <message>
         <source>Cannot generate valid passphrases because the wordlist is too short</source>
         <translation type="unfinished"></translation>
@@ -9856,29 +9876,6 @@ This option is deprecated, use --set-key-file instead.</source>
         <translation type="unfinished"></translation>
     </message>
 </context>
-<context>
-    <name>ShortcutSettingsWidget</name>
-    <message>
-        <source>Double click an action to change its shortcut</source>
-        <translation type="unfinished"></translation>
-    </message>
-    <message>
-        <source>Shortcut Conflict</source>
-        <translation type="unfinished"></translation>
-    </message>
-    <message>
-        <source>Filter...</source>
-        <translation type="unfinished"></translation>
-    </message>
-    <message>
-        <source>Shortcut %1 conflicts with &apos;%2&apos;. Overwrite shortcut?</source>
-        <translation type="unfinished"></translation>
-    </message>
-    <message>
-        <source>Reset Shortcuts</source>
-        <translation type="unfinished"></translation>
-    </message>
-</context>
 <context>
     <name>TagModel</name>
     <message>

diff --git a/src/gui/ShortcutSettingsPage.cpp b/src/gui/ShortcutSettingsPage.cpp
@@ -23,7 +23,6 @@
 #include "gui/MessageBox.h"
 #include "gui/widgets/ShortcutWidget.h"
 
-#include <QAbstractButton>
 #include <QDebug>
 #include <QDialog>
 #include <QDialogButtonBox>
@@ -81,7 +80,6 @@ class KeySequenceDialog final : public QDialog
         }
     }
 
-private:
     bool m_restoreDefault = false;
     ShortcutWidget* const m_keySeqEdit;
     QDialogButtonBox* const m_btnBox;
@@ -94,15 +92,15 @@ class ShortcutSettingsWidget final : public QWidget
         : QWidget(parent)
         , m_tableView(new QTableView(this))
         , m_filterLineEdit(new QLineEdit(this))
-        , m_resetShortcutsButton(new QPushButton(tr("Reset Shortcuts"), this))
+        , m_resetShortcutsButton(new QPushButton(QObject::tr("Reset Shortcuts"), this))
     {
         auto h = new QHBoxLayout();
         h->addWidget(m_filterLineEdit);
         h->addWidget(m_resetShortcutsButton);
         h->setStretch(0, 1);
 
         auto l = new QVBoxLayout(this);
-        l->addWidget(new QLabel(tr("Double click an action to change its shortcut")));
+        l->addWidget(new QLabel(QObject::tr("Double click an action to change its shortcut")));
         l->addLayout(h);
         l->addWidget(m_tableView);
 
@@ -113,7 +111,7 @@ class ShortcutSettingsWidget final : public QWidget
         m_proxy.setFilterCaseSensitivity(Qt::CaseInsensitive);
         m_proxy.setSourceModel(&m_model);
 
-        m_filterLineEdit->setPlaceholderText(tr("Filter..."));
+        m_filterLineEdit->setPlaceholderText(QObject::tr("Filter..."));
         connect(m_filterLineEdit, &QLineEdit::textChanged, &m_proxy, &QSortFilterProxyModel::setFilterFixedString);
 
         connect(m_resetShortcutsButton, &QPushButton::clicked, this, [this]() {
@@ -227,8 +225,8 @@ class ShortcutSettingsWidget final : public QWidget
 
             auto ans = MessageBox::question(
                 this,
-                tr("Shortcut Conflict"),
-                tr("Shortcut %1 conflicts with '%2'. Overwrite shortcut?").arg(conflictSeq, conflictName),
+                QObject::tr("Shortcut Conflict"),
+                QObject::tr("Shortcut %1 conflicts with '%2'. Overwrite shortcut?").arg(conflictSeq, conflictName),
                 MessageBox::Overwrite | MessageBox::Discard,
                 MessageBox::Discard);
             if (ans == MessageBox::Discard) {