The &-shortcut from another action is not set until the action has been shown at...

[dolphin.git] / src / views / selectiontoggle.cpp

diff --git a/src/views/selectiontoggle.cpp b/src/views/selectiontoggle.cpp
index cbd273b11924f577db2a899a286775bc9987a782..c1272fa0f764385131dc65e1df264e4e8497adc0 100644 (file)
--- a/src/views/selectiontoggle.cpp
+++ b/src/views/selectiontoggle.cpp
@@ -32,11 +32,12 @@
 #include <QTimer>
 #include <QTimeLine>
 
 #include <QTimer>
 #include <QTimeLine>
 

+#include <kdebug.h>
+

 SelectionToggle::SelectionToggle(QWidget* parent) :
     QAbstractButton(parent),
     m_isHovered(false),
     m_leftMouseButtonPressed(false),
 SelectionToggle::SelectionToggle(QWidget* parent) :
     QAbstractButton(parent),
     m_isHovered(false),
     m_leftMouseButtonPressed(false),

-    m_appliedArrowCursor(false),

     m_fadingValue(0),
     m_margin(0),
     m_icon(),
     m_fadingValue(0),
     m_margin(0),
     m_icon(),


@@ -106,25 +107,12 @@ void SelectionToggle::setVisible(bool visible)
 
 bool SelectionToggle::eventFilter(QObject* obj, QEvent* event)
 {
 
 bool SelectionToggle::eventFilter(QObject* obj, QEvent* event)
 {

-    if (obj == parent()) {
-        switch (event->type()) {
-        case QEvent::Leave:
-            hide();
-            break;
-
-        case QEvent::MouseMove:
-            if (m_leftMouseButtonPressed) {
-                // Don't forward mouse move events to the viewport,
-                // otherwise a rubberband selection will be shown when
-                // clicking on the selection toggle and moving the mouse
-                // above the viewport.
-                return true;
-            }
-            break;
-
-        default:
-            break;
-        }
+    if ((obj == parent()) && (event->type() == QEvent::MouseMove) && m_leftMouseButtonPressed) {
+        // Don't forward mouse move events to the viewport,
+        // otherwise a rubberband selection will be shown when
+        // clicking on the selection toggle and moving the mouse
+        // above the viewport.
+        return true;

     }
 
     return QAbstractButton::eventFilter(obj, event);
     }
 
     return QAbstractButton::eventFilter(obj, event);


@@ -134,19 +122,6 @@ void SelectionToggle::enterEvent(QEvent* event)
 {
     QAbstractButton::enterEvent(event);
 
 {
     QAbstractButton::enterEvent(event);
 

-    if (!m_appliedArrowCursor) {
-        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
     // it immediately without fading timer
     m_isHovered = true;
     // if the mouse cursor is above the selection toggle, display
     // it immediately without fading timer
     m_isHovered = true;


@@ -163,13 +138,6 @@ void SelectionToggle::leaveEvent(QEvent* event)
 {
     QAbstractButton::leaveEvent(event);
 
 {
     QAbstractButton::leaveEvent(event);
 

-    if (m_appliedArrowCursor) {
-        // Reset the cursor asynchronously. See SelectionToggle::enterEvent()
-        // for a more information.
-        m_appliedArrowCursor = false;
-        QTimer::singleShot(0, this, SLOT(restoreCursor()));
-    }
-

     m_isHovered = false;
     update();
 }
     m_isHovered = false;
     update();
 }


@@ -245,16 +213,6 @@ void SelectionToggle::refreshIcon()
     setIconOverlay(isChecked());
 }
 
     setIconOverlay(isChecked());
 }
 

-void SelectionToggle::applyArrowCursor()
-{
-    QApplication::setOverrideCursor(QCursor(Qt::ArrowCursor));
-}
-
-void SelectionToggle::restoreCursor()
-{
-    QApplication::restoreOverrideCursor();
-}
-

 void SelectionToggle::startFading()
 {
     Q_ASSERT(m_fadingTimeLine == 0);
 void SelectionToggle::startFading()
 {
     Q_ASSERT(m_fadingTimeLine == 0);