From 06c35026d650e2d35f4007012879b37672508e32 Mon Sep 17 00:00:00 2001 From: Berthold Stoeger Date: Fri, 2 Oct 2020 18:47:05 +0200 Subject: desktop: fix paging through dive list with page-up key, etc In the dive list we have horrible code, which intercepts all events to save the selection before/after the event. This was necessary because we couldn't get Qt's selection data flow under control. This means intercepting all events that can change the selection. The page-up, page-down, home and end keys were forgotten. Add these cases. Fixes #2957. Signed-off-by: Berthold Stoeger --- desktop-widgets/divelistview.cpp | 19 +++++++++++++++---- 1 file changed, 15 insertions(+), 4 deletions(-) (limited to 'desktop-widgets/divelistview.cpp') diff --git a/desktop-widgets/divelistview.cpp b/desktop-widgets/divelistview.cpp index 900114d05..390c78216 100644 --- a/desktop-widgets/divelistview.cpp +++ b/desktop-widgets/divelistview.cpp @@ -448,10 +448,21 @@ void DiveListView::mouseReleaseEvent(QMouseEvent *event) void DiveListView::keyPressEvent(QKeyEvent *event) { - // Hook into cursor-up and cursor-down events and update selection if necessary. - // See comment in mouseReleaseEvent() - if (event->key() != Qt::Key_Down && event->key() != Qt::Key_Up) - return QTreeView::keyPressEvent(event); + // Hook into key events that change the selection (i.e. cursor-up, cursor-down, + // page-up, page-down, home and end) and update selection if necessary. + // See comment in mouseReleaseEvent(). + switch (event->key()) { + case Qt::Key_Up: + case Qt::Key_Down: + case Qt::Key_PageUp: + case Qt::Key_PageDown: + case Qt::Key_Home: + case Qt::Key_End: + break; + default: + return QTreeView::keyPressEvent(event); + } + QModelIndexList selectionBefore = selectionModel()->selectedRows(); QTreeView::keyPressEvent(event); QModelIndexList selectionAfter = selectionModel()->selectedRows(); -- cgit v1.2.3-70-g09d2