Review Request 127544: Remove notifications from plasmoid, if the notifications plugin gets disabled

David Kahles david.kahles96 at gmail.com
Fri Apr 1 13:47:08 UTC 2016


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

Review request for kdeconnect.


Repository: kdeconnect-kde


Description
-------

Therefore create a component which checks whether the plugin is loaded,
and exposes this information to the DeviceDelegate. The code is mostly
copied from FindMyPhone.qml.


Diffs
-----

  plasmoid/package/contents/ui/DeviceDelegate.qml 34dfb1952ed6973c8b3ce5b96fc4799ba37610d7 
  plasmoid/package/contents/ui/NotificationsPluginChecker.qml PRE-CREATION 

Diff: https://git.reviewboard.kde.org/r/127544/diff/


Testing
-------

If i disabled the notification plugin, the notifications stayed in the plasmoid though they were purged in the server, because they were cached.
With this patch, the notifications disappear immediately.


Thanks,

David Kahles

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20160401/7385f1a5/attachment.html>


More information about the KDEConnect mailing list