From 2f4a36f0ff8285e80de2c4b4670b522261963190 Mon Sep 17 00:00:00 2001 From: ShaopengLin Date: Tue, 4 Jun 2024 23:10:18 -0400 Subject: [PATCH] Refactored ContentManager click signal handler Handler is now a slot function instead of a lambda. --- src/contentmanagerview.cpp | 25 ++++++++++++++----------- src/contentmanagerview.h | 1 + 2 files changed, 15 insertions(+), 11 deletions(-) diff --git a/src/contentmanagerview.cpp b/src/contentmanagerview.cpp index 24ba1f9..75847d3 100644 --- a/src/contentmanagerview.cpp +++ b/src/contentmanagerview.cpp @@ -19,17 +19,7 @@ ContentManagerView::ContentManagerView(QWidget *parent) loader = new KiwixLoader(mp_ui->loading); mp_ui->stackedWidget->setCurrentIndex(0); - connect(mp_ui->m_view, &QTreeView::clicked, [=](QModelIndex index) { - if (index.column() == (mp_ui->m_view->model()->columnCount() - 1)) - return; - - auto zeroColIndex = index.siblingAtColumn(0); - if (mp_ui->m_view->isExpanded(zeroColIndex)) { - mp_ui->m_view->collapse(zeroColIndex); - } else { - mp_ui->m_view->expand(zeroColIndex); - } - }); + connect(mp_ui->m_view, &QTreeView::clicked, this, &ContentManagerView::onClicked); } ContentManagerView::~ContentManagerView() @@ -46,3 +36,16 @@ void ContentManagerView::showLoader(bool show) loader->stopAnimation(); } } + +void ContentManagerView::onClicked(QModelIndex index) +{ + if (index.column() == (mp_ui->m_view->model()->columnCount() - 1)) + return; + + auto zeroColIndex = index.siblingAtColumn(0); + if (mp_ui->m_view->isExpanded(zeroColIndex)) { + mp_ui->m_view->collapse(zeroColIndex); + } else { + mp_ui->m_view->expand(zeroColIndex); + } +} diff --git a/src/contentmanagerview.h b/src/contentmanagerview.h index 68017a5..790faf3 100644 --- a/src/contentmanagerview.h +++ b/src/contentmanagerview.h @@ -20,6 +20,7 @@ public: public slots: void showLoader(bool show); + void onClicked(QModelIndex index); private: Ui::contentmanagerview *mp_ui;