diff options
| author | Ahmad Samir <[email protected]> | 2020-09-20 18:38:29 +0200 |
|---|---|---|
| committer | Ahmad Samir <[email protected]> | 2020-09-20 18:38:29 +0200 |
| commit | d0c71a1435bc9d67475c5525c3641bd4a13c61f0 (patch) | |
| tree | f2f0d939799fe96897b23fbe352b846a17c3afc6 /src/kitemviews/kitemlistcontroller.cpp | |
| parent | 827b79f5c85118730db3b0642245f09071703c97 (diff) | |
Fix mouse back/forward buttons
We need to emit mouseButtonPressed (now in the onPress method) before the
check needed to not select items on back/foward mouse button press events.
Diffstat (limited to 'src/kitemviews/kitemlistcontroller.cpp')
| -rw-r--r-- | src/kitemviews/kitemlistcontroller.cpp | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/src/kitemviews/kitemlistcontroller.cpp b/src/kitemviews/kitemlistcontroller.cpp index 650bf6286..1733eeab5 100644 --- a/src/kitemviews/kitemlistcontroller.cpp +++ b/src/kitemviews/kitemlistcontroller.cpp @@ -550,17 +550,19 @@ bool KItemListController::mousePressEvent(QGraphicsSceneMouseEvent* event, const m_pressedMousePos = transform.map(event->pos()); m_pressedIndex = m_view->itemAt(m_pressedMousePos); - if (event->buttons() & (Qt::BackButton | Qt::ForwardButton)) { - // Do not select items when clicking the back/forward buttons, see - // https://bugs.kde.org/show_bug.cgi?id=327412. - return true; - } - const Qt::MouseButtons buttons = event->buttons(); + if (!onPress(event->screenPos(), event->pos(), event->modifiers(), buttons)) { startRubberBand(); return false; } + + if (buttons & (Qt::BackButton | Qt::ForwardButton)) { + // Do not select items when clicking the back/forward buttons, see + // https://bugs.kde.org/show_bug.cgi?id=327412. + return true; + } + return true; } |
