Enable changing the selected filter using the arrow keys

#295
powertab · Jun 18, 2020 · 32222fd · 32222fd
1 parent 0152c33
commit 32222fd
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/source/dialogs/viewfilterdialog.cpp b/source/dialogs/viewfilterdialog.cpp
@@ -43,8 +43,11 @@ void ViewFilterDialog::setPresenter(ViewFilterPresenter *presenter)
     connect(ui->removeFilterButton, &QToolButton::clicked, [&]() {
         myPresenter->removeSelectedFilter();
     });
-    connect(ui->filterList, &QListWidget::itemClicked, [&](QListWidgetItem *) {
-        myPresenter->selectFilter(ui->filterList->currentRow());
+    connect(ui->filterList, &QListWidget::currentRowChanged, [&](int row) {
+        // Don't send selection events when the filter list is cleared during
+        // update().
+        if (row >= 0)
+            myPresenter->selectFilter(row);
     });
 
     connect(ui->nameLineEdit, &QLineEdit::textEdited, [&](const QString &s) {