[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdeplasma-addons/applets/frame
From:       Anne-Marie Mahfouf <annma () kde ! org>
Date:       2010-10-10 15:47:32
Message-ID: 20101010154732.A4A173E1F1 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1184499 by annma:

when implementing the slideshow arrows, we forgot that a drop event also triggers a \
hover event. After one dropped a folder and then a static picture, the arrows stayed \
on the static picture. this fix ensures the arrows are properly hidden


 M  +5 -1      frame.cpp  


--- trunk/KDE/kdeplasma-addons/applets/frame/frame.cpp #1184498:1184499
@@ -633,6 +633,9 @@
 
 void Frame::dropEvent(QGraphicsSceneDragDropEvent *event)
 {
+    if (m_slideFrame) {
+        m_slideFrame->hide();
+    }
     KUrl droppedUrl = (KUrl::List::fromMimeData(event->mimeData())).at(0);
     kDebug() << "dropped URL" << droppedUrl.url();
     if (droppedUrl.protocol() == "desktop") {
@@ -664,8 +667,8 @@
 
 void Frame::hoverEnterEvent(QGraphicsSceneHoverEvent *event)
 {
+    checkSlideFrame();
     if (m_slideShow) {
-        checkSlideFrame();
         m_slideFrame->show();
     }
     Applet::hoverEnterEvent(event);
@@ -683,6 +686,7 @@
 void Frame::checkSlideFrame()
 {
     if (m_slideFrame) {
+        m_slideFrame->hide();
         return;
     }
     m_slideFrame = new Plasma::Frame( this );


[prev in list] [next in list] [prev in thread] [next in thread]