feat: check conflict shortcuts before saving

app/settings.cpp

@@ -133,23 +133,31 @@ void Settings::applyUserShortcuts(QWidget *widget)
 bool Settings::setShortcutsForAction(QWidget *widget, const QString &objectName,
                                      QList<QKeySequence> shortcuts, bool writeConfig)
 {
-    bool result = false;
+    QAction * targetAction = nullptr;
     for (QAction * action : widget->actions()) {
         if (action->objectName() == objectName) {
-            action->setShortcuts(shortcuts);
+            targetAction = action;
-            result = true;
+        } else {
-            break;
+            for (const QKeySequence & shortcut : std::as_const(shortcuts)) {
+                if (action->shortcuts().contains(shortcut)) {
+                    return false;
+                }
+            }
         }
     }
 
-    if (result && writeConfig) {
+    if (targetAction) {
+        targetAction->setShortcuts(shortcuts);
+    }
+
+    if (targetAction && writeConfig) {
         m_qsettings->beginGroup("shortcuts");
         m_qsettings->setValue(objectName, QVariant::fromValue(shortcuts));
         m_qsettings->endGroup();
         m_qsettings->sync();
     }
 
-    return result;
+    return true;
 }
 
 #if defined(FLAG_PORTABLE_MODE_SUPPORT) && defined(Q_OS_WIN)

app/settingsdialog.cpp

@@ -15,6 +15,7 @@
 #include <QScrollArea>
 #include <QSplitter>
 #include <QStringListModel>
+#include <QMessageBox>
 
 SettingsDialog::SettingsDialog(QWidget *parent)
     : QDialog(parent)
@@ -60,9 +61,14 @@ SettingsDialog::SettingsDialog(QWidget *parent)
             shortcutEditorSplitter->setSizes({shortcutEditorSplitter->height(), 1});
             oldEditor->deleteLater();
         });
-        connect(shortcutEdit, &ShortcutEdit::shortcutsChanged, this, [=](){
+        connect(shortcutEdit, &ShortcutEdit::applyShortcutsRequested, this, [=](QList<QKeySequence> newShortcuts){
-            Settings::instance()->setShortcutsForAction(parent, shortcutEdit->objectName().mid(9),
+            bool succ = Settings::instance()->setShortcutsForAction(parent, shortcutEdit->objectName().mid(9),
-                                                        shortcutEdit->shortcuts());
+                                                                    newShortcuts);
+            if (!succ) {
+                QMessageBox::warning(this, tr("Failed to set shortcuts"),
+                                     tr("Please check if shortcuts are duplicated with existing shortcuts."));
+            }
+            shortcutEdit->setShortcuts(action->shortcuts());
         });
     }
 

app/shortcutedit.cpp

@@ -33,6 +33,7 @@ ShortcutEditor::ShortcutEditor(ShortcutEdit * shortcutEdit, QWidget * parent)
             reloadShortcuts();
         }
     });
+    connect(shortcutEdit, &ShortcutEdit::shortcutsChanged, this, &ShortcutEditor::reloadShortcuts);
 
     setSizePolicy(QSizePolicy::Preferred, QSizePolicy::Minimum);
 
@@ -81,12 +82,11 @@ void ShortcutEditor::applyShortcuts()
 {
     QList<QKeySequence> shortcuts;
     for (const QKeySequenceEdit * keyseqEdit : m_keySequenceEdits) {
-        if (!keyseqEdit->keySequence().isEmpty()) {
+        if (!keyseqEdit->keySequence().isEmpty() && !shortcuts.contains(keyseqEdit->keySequence())) {
             shortcuts.append(keyseqEdit->keySequence());
         }
     }
-    m_shortcutEdit->setShortcuts(shortcuts);
+    emit m_shortcutEdit->applyShortcutsRequested(shortcuts);
-    reloadShortcuts();
 }
 
 // ----------------------------------------

app/shortcutedit.h

@@ -46,6 +46,7 @@ public:
 signals:
     void shortcutsChanged();
     void editButtonClicked();
+    void applyShortcutsRequested(QList<QKeySequence> newShortcuts);
 
 private:
     QList<QKeySequence> m_shortcuts;