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

List:       kde-kimageshop
Subject:    Review Patch for cursor on popup palette
From:       Sven Langkamp <sven.langkamp () gmail ! com>
Date:       2010-03-03 10:58:56
Message-ID: 478b087a1003030258i7fbb50c7x90f516b7d051e628 () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Hi,

I have attached a patch for bug https://bugs.kde.org/show_bug.cgi?id=228976
It will override the cursor when the palette is shown and restore it on
hide. I'm not sure if it's the correct way to fix that bug so posting for
review.



Sven

[Attachment #5 (text/html)]

Hi,<br><br>I have attached a patch for bug <a \
href="https://bugs.kde.org/show_bug.cgi?id=228976">https://bugs.kde.org/show_bug.cgi?id=228976</a><br>It \
will override the cursor when the palette is shown and restore it on hide. I&#39;m \
not sure if it&#39;s the correct way to fix that bug so posting for review.<br> \
<br><br><br>Sven<br>

--001485f7d7826368240480e35efc--


["popuppalettecursor.diff" (text/x-patch)]

Index: kis_popup_palette.cpp
===================================================================
--- kis_popup_palette.cpp	(Revision 1098216)
+++ kis_popup_palette.cpp	(Arbeitskopie)
@@ -127,6 +127,10 @@
         emit sigEnableChangeFGColor(!b);
     else
         emit sigTriggerTimer();
+    if(b)
+         QApplication::setOverrideCursor(Qt::ArrowCursor);
+    else
+        QApplication::restoreOverrideCursor();
     setVisible(b);
 }
 


_______________________________________________
kimageshop mailing list
kimageshop@kde.org
https://mail.kde.org/mailman/listinfo/kimageshop


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