smw/deskflow
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

refactor: ActionDialog, set the current screen combo without the need to track how many items are in the combobox with an additional int variable

Browse source
This commit is contained in:
sithlord48 2025-01-04 19:23:42 -05:00 committed by Nick Bolton
parent 1129303254
commit 096ab3b8ed
1 changed files with 1 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/apps/deskflow-gui/dialogs/ActionDialog.cpp
View file

@ -64,7 +64,6 @@ ActionDialog::ActionDialog(QWidget *parent, const ServerConfig &config, Hotkey &
ui->m_pGroupBoxScreens->setChecked(m_Action.haveScreens()); ui->m_pGroupBoxScreens->setChecked(m_Action.haveScreens());
int idx = 0;
for (const Screen &screen : config.screens()) { for (const Screen &screen : config.screens()) {
if (screen.isNull()) if (screen.isNull())
continue; continue;
@ -75,9 +74,7 @@ ActionDialog::ActionDialog(QWidget *parent, const ServerConfig &config, Hotkey &
ui->m_pComboSwitchToScreen->addItem(screen.name()); ui->m_pComboSwitchToScreen->addItem(screen.name());
if (screen.name() == m_Action.switchScreenName()) if (screen.name() == m_Action.switchScreenName())
ui->m_pComboSwitchToScreen->setCurrentIndex(idx); ui->m_pComboSwitchToScreen->setCurrentIndex(ui->m_pComboSwitchToScreen->count() - 1);
idx++;
} }
} }