Review Request 124037: Allow to cancel critical battery timer

Sebastian Kügler sebas at kde.org
Mon Jun 8 17:12:11 UTC 2015


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

Ship it!


I saw Vishesh running with the laptop in his hands here, so he's in danger of also running into this bone-breaking problem.

Perhaps we can avoid the repitition of the title in the maintext and make it fit better this way?

- Sebastian Kügler


On June 7, 2015, 7:09 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124037/
> -----------------------------------------------------------
> 
> (Updated June 7, 2015, 7:09 p.m.)
> 
> 
> Review request for Plasma, Solid and KDE Usability.
> 
> 
> Repository: powerdevil
> 
> 
> Description
> -------
> 
> To prevent David from eventually breaking his bones tumbling down the stairs rushing for his AC adapter, this adds a "Cancel" button to the battery critical notification that allows to cancel the timeout for automatic suspend/shutdown.
> 
> 
> Diffs
> -----
> 
>   daemon/powerdevilcore.h 50e6a50 
>   daemon/powerdevilcore.cpp e90a960 
> 
> Diff: https://git.reviewboard.kde.org/r/124037/diff/
> 
> 
> Testing
> -------
> 
> Got a "Cancel" button only when it was configured to do something, clicking the button from both popup and history cancelled the timeout. (Unfortunately the notification doesn't fit into the popup at all now)
> 
> 
> File Attachments
> ----------------
> 
> Notification with close button
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/06/07/f99c0164-4436-462f-974b-ddc5df0ce500__powerdevilcanceltimeout.png
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150608/bbfb6a87/attachment-0001.html>


More information about the Plasma-devel mailing list