playground/base/plasma/applets/systray-refactor

Jason Stubbs jasonbstubbs at gmail.com
Sun Oct 5 17:08:31 CEST 2008


SVN commit 868140 by jstubbs:

Overriding the default hoverEnterEvent and hoverLeaveEvent handlers as they
call update(), which is not required.

There is still a pointless update when the mouse cursor enters the panel,
though. It appears to be coming from Plasma::Containment and is causing
everything to repaint. There's an unexplained FIXME that's related to
Qt-4.4 that needs to be checked in order to fix this issue.

CCMAIL: plasma-devel at kde.org


 M  +0 -7      TODO  
 M  +3 -0      ui/applet.h  


--- trunk/playground/base/plasma/applets/systray-refactor/TODO #868139:868140
@@ -1,10 +1,3 @@
-* hoverEnterEvent paint problem
-  - Should Applet really be doing setAcceptHoverEvents(true) ?
-    - Unrelated, but should Containment be doing it either?
-  - Sort out the above and/or do setAcceptHoverEvents(false) or reimplement hoverEnterEvent()
-
-
-
 * Positioning of the extender is wrong. (Notifications are drawn part off screen and behind systray icons)
 * Misbehavior of certain applets within the plasmoid protocol or vice-versa. (Still needs investigating)
 * Notifications aren't grouped by application. (Not critical but preferred)
--- trunk/playground/base/plasma/applets/systray-refactor/ui/applet.h #868139:868140
@@ -49,6 +49,9 @@
     void createConfigurationInterface(KConfigDialog *parent);
     void initExtenderItem(Plasma::ExtenderItem *extenderItem);
 
+    void hoverEnterEvent(QGraphicsSceneHoverEvent *event) { Q_UNUSED(event); }
+    void hoverLeaveEvent(QGraphicsSceneHoverEvent *event) { Q_UNUSED(event); }
+
 private slots:
     void configAccepted();
     void propogateSizeHintChange(Qt::SizeHint which);


More information about the Plasma-devel mailing list