Review Request: Device notifier popup did not hide after the intended timeout of 7.5 s.

Simon Persson simonpersson1 at gmail.com
Tue Oct 11 10:18:10 UTC 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102824/
-----------------------------------------------------------

Review request for Plasma, Giulio Camuffo and Jacopo De Simoi.


Description
-------

This is my first fix for plasma, maybe there's a reason to emit plasma::applet::activated() that I don't know. Removing it fixes the problem and I can't see any regression.
I also needed to remove activated() signal from NotifierDialog, I consider this one to be unnecessary since there is an eventfilter that on QEvent::GraphicsSceneHoverMove triggers the popup to be open for 7.5 s more. In other words, the popup will now close after 7.5 s of inactive mouse if it was shown as a result of plugging in a device... which I guess was the intended behavior.

Couldn't actually find any open bug report for this. (!)


Diffs
-----

  plasma/generic/applets/devicenotifier/notifierdialog.cpp dff38d9 
  plasma/generic/applets/devicenotifier/notifierdialog.h 8a03fc5 
  plasma/generic/applets/devicenotifier/devicenotifier.cpp b9dfce5 

Diff: http://git.reviewboard.kde.org/r/102824/diff/diff


Testing
-------


Thanks,

Simon Persson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20111011/01fc5067/attachment.html>


More information about the Plasma-devel mailing list