mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2024-12-01 06:30:26 +00:00
Code Issues Projects Releases Packages Wiki Activity

Clean up divelist dive selection

Browse source 
Since we have a "selectDive()" method, let's just use it rather than
opencoding it.  Similarly for "unselectDives()".

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Linus Torvalds 2013-06-05 17:28:22 +09:00 committed by Dirk Hohndel
parent 4d937051ca
commit 4b405caa2c
1 changed files with 4 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
qt-ui/divelistview.cpp
View file

@ -91,7 +91,7 @@ void DiveListView::headerClicked(int i)
QItemSelection oldSelection = selectionModel()->selection();
QList<struct dive*> currentSelectedDives;
DiveTripModel::Layout newLayout;
bool scrolled = false;
bool first = true;
newLayout = i == (int) TreeItemDT::NR ? DiveTripModel::TREE : DiveTripModel::LIST;
@ -104,7 +104,7 @@ void DiveListView::headerClicked(int i)
currentSelectedDives.push_back(d);
}
selectionModel()->clearSelection();
unselectDives();
/* No layout change? Just re-sort, and scroll to first selection, making sure all selections are expanded */
if (currentLayout == newLayout) {
@ -119,19 +119,8 @@ void DiveListView::headerClicked(int i)
// repopulat the selections.
Q_FOREACH(struct dive *d, currentSelectedDives) {
QModelIndexList match = m->match(m->index(0,0), TreeItemDT::NR, d->number, 1, Qt::MatchRecursive);
QModelIndex idx = match.first();
if (newLayout == DiveTripModel::TREE) {
QModelIndex parent = idx.parent();
if (parent.isValid())
expand(parent);
}
selectionModel()->select( idx, QItemSelectionModel::Select | QItemSelectionModel::Rows);
if (!scrolled) {
scrollTo(idx, PositionAtCenter);
scrolled = true;
}
selectDive(d, first);
first = false;
}
}