X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/blobdiff_plain/e2b844437e9308a04ddf043ae9d2f5c8ffef97b0..6c8c052b3ffa628fd99f4e4d726017e2fc8e0e1d:/src/views/selectiontoggle.cpp diff --git a/src/views/selectiontoggle.cpp b/src/views/selectiontoggle.cpp index 3ab40ebdc..cbd273b11 100644 --- a/src/views/selectiontoggle.cpp +++ b/src/views/selectiontoggle.cpp @@ -135,8 +135,16 @@ void SelectionToggle::enterEvent(QEvent* event) QAbstractButton::enterEvent(event); if (!m_appliedArrowCursor) { - QApplication::setOverrideCursor(QCursor(Qt::ArrowCursor)); m_appliedArrowCursor = true; + // Apply the arrow asynchronously. This is required for + // the following usecase: + // 1. Cursor is above the viewport left beside an item + // 2. Cursor is moved above the item, so that the selection-toggle + // and the item are entered equally. + // In this situation it is not defined who gets the enter-event first. + // As the selection-toggle is above the item, it should overwrite possible + // cursor changes done by the item. + QTimer::singleShot(0, this, SLOT(applyArrowCursor())); } // if the mouse cursor is above the selection toggle, display @@ -156,8 +164,10 @@ void SelectionToggle::leaveEvent(QEvent* event) QAbstractButton::leaveEvent(event); if (m_appliedArrowCursor) { - QApplication::restoreOverrideCursor(); + // Reset the cursor asynchronously. See SelectionToggle::enterEvent() + // for a more information. m_appliedArrowCursor = false; + QTimer::singleShot(0, this, SLOT(restoreCursor())); } m_isHovered = false; @@ -235,6 +245,16 @@ void SelectionToggle::refreshIcon() setIconOverlay(isChecked()); } +void SelectionToggle::applyArrowCursor() +{ + QApplication::setOverrideCursor(QCursor(Qt::ArrowCursor)); +} + +void SelectionToggle::restoreCursor() +{ + QApplication::restoreOverrideCursor(); +} + void SelectionToggle::startFading() { Q_ASSERT(m_fadingTimeLine == 0);