----------------------------------------------------------- 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
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel