Review Request 121692: Implement a plugin for the inhibition of the screensaver when a device is connected

Albert Vaca Cintora albertvaka at gmail.com
Mon Dec 29 07:55:16 UTC 2014


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

Ship it!


Perfect, thank you! I will merge it now.

- Albert Vaca Cintora


On des. 29, 2014, 8:46 a.m., Pramod Dematagoda wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121692/
> -----------------------------------------------------------
> 
> (Updated des. 29, 2014, 8:46 a.m.)
> 
> 
> Review request for kdeconnect and Albert Vaca Cintora.
> 
> 
> Repository: kdeconnect-kde
> 
> 
> Description
> -------
> 
> This plugin serves to inhibit the screensaver when enabled for a given device and that device is connected to kdeconnect. The inhibition is lifted when the plugin is either disabled or the device disconnects from kdeconnect.
> 
> 
> Diffs
> -----
> 
>   plugins/screensaver-inhibit/README PRE-CREATION 
>   plugins/screensaver-inhibit/kdeconnect_screensaver_inhibit.desktop PRE-CREATION 
>   plugins/screensaver-inhibit/screensaverinhibitplugin.h PRE-CREATION 
>   plugins/screensaver-inhibit/screensaverinhibitplugin.cpp PRE-CREATION 
>   plugins/CMakeLists.txt de1131d 
>   plugins/screensaver-inhibit/CMakeLists.txt PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/121692/diff/
> 
> 
> Testing
> -------
> 
> Simple testing where a phone with kdeconnect is connected to a laptop with the proposed patch and it is observed that the screensaver does not start with the phone connected and the plugin enabled. The screensaver does start with the plugin either disabled or when the phone is disconnected.
> 
> 
> Thanks,
> 
> Pramod Dematagoda
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20141229/16c857d5/attachment.html>


More information about the KDEConnect mailing list