From ade0d8e758bd925f919d172c0293eb160502e0fd Mon Sep 17 00:00:00 2001 From: Berthold Stoeger Date: Mon, 29 Oct 2018 15:06:23 +0100 Subject: [PATCH] Dive list: connect header-signal in constructor The QHeaderView::sectionPressed() signal was connected everytime the list-view was reset. Likewise, setSectionsClickable() was set to true everythime the list-view was reset. Once in the constructor is enough. Signed-off-by: Berthold Stoeger --- desktop-widgets/divelistview.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/desktop-widgets/divelistview.cpp b/desktop-widgets/divelistview.cpp index 30364a8eb..9c5ea8b1c 100644 --- a/desktop-widgets/divelistview.cpp +++ b/desktop-widgets/divelistview.cpp @@ -47,6 +47,8 @@ DiveListView::DiveListView(QWidget *parent) : QTreeView(parent), mouseClickSelec header()->setStretchLastSection(true); header()->setSortIndicatorShown(true); + header()->setSectionsClickable(true); + connect(header(), &QHeaderView::sectionPressed, this, &DiveListView::headerClicked); installEventFilter(this); @@ -495,9 +497,6 @@ void DiveListView::reload(DiveTripModel::Layout layout, bool forceSort) else currentLayout = layout; - header()->setSectionsClickable(true); - connect(header(), SIGNAL(sectionPressed(int)), this, SLOT(headerClicked(int)), Qt::UniqueConnection); - MultiFilterSortModel::instance()->setLayout(layout); if (!forceSort)