From 84c223486946d0abb1af7e8526ea9f1321c200cf Mon Sep 17 00:00:00 2001 From: sithlord48 Date: Thu, 24 Jul 2025 20:16:58 -0400 Subject: [PATCH] refactor: ActionDialog use proper ActionDialog::itemToggled in place of lambda --- src/lib/gui/dialogs/ActionDialog.cpp | 9 ++++++--- src/lib/gui/dialogs/ActionDialog.h | 1 + 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/src/lib/gui/dialogs/ActionDialog.cpp b/src/lib/gui/dialogs/ActionDialog.cpp index 9649681ff..16f06950c 100644 --- a/src/lib/gui/dialogs/ActionDialog.cpp +++ b/src/lib/gui/dialogs/ActionDialog.cpp @@ -27,9 +27,7 @@ ActionDialog::ActionDialog(QWidget *parent, const ServerConfig &config, Hotkey & connect( ui->comboActionType, QOverload::of(&QComboBox::currentIndexChanged), this, &ActionDialog::actionTypeChanged ); - connect(ui->listScreens, &QListWidget::itemChanged, this, [&] { - ui->buttonBox->button(QDialogButtonBox::Ok)->setEnabled(canSave()); - }); + connect(ui->listScreens, &QListWidget::itemChanged, this, &ActionDialog::itemToggled); connect(ui->buttonBox, &QDialogButtonBox::accepted, this, &ActionDialog::accept); connect(ui->buttonBox, &QDialogButtonBox::rejected, this, &ActionDialog::reject); @@ -117,6 +115,11 @@ void ActionDialog::keySequenceChanged() ui->buttonBox->button(QDialogButtonBox::Ok)->setEnabled(canSave()); } +void ActionDialog::itemToggled() const +{ + ui->buttonBox->button(QDialogButtonBox::Ok)->setEnabled(canSave()); +} + void ActionDialog::actionTypeChanged(int index) { ui->buttonBox->button(QDialogButtonBox::Ok)->setEnabled(canSave()); diff --git a/src/lib/gui/dialogs/ActionDialog.h b/src/lib/gui/dialogs/ActionDialog.h index 1d5de5108..f50c7b261 100644 --- a/src/lib/gui/dialogs/ActionDialog.h +++ b/src/lib/gui/dialogs/ActionDialog.h @@ -44,6 +44,7 @@ protected Q_SLOTS: private: void updateSize(); void keySequenceChanged(); + void itemToggled() const; void actionTypeChanged(int index); bool isKeyAction(int index) const; bool canSave() const;