Review Request 127263: Add Wireless Energy Saving action

[Thomas Pfeiffer]

> On March 9, 2016, 2:47 p.m., Thomas Pfeiffer wrote:
> > Thank you for the screenshot!
> > It looks good to me, the only thing I'd change is relabeling "Manual configuration" to "Leave unchanged" (and changing functionality if that isn't what it does atm) and putting that as the first option and making it the default.
> 
> Jan Grulich wrote:
>     That's what it does, it basically does nothing and leaves this to the user to manage his wifi/wwan/bt through plasma-nm or the bluetooth applet.
> 
> Thomas Pfeiffer wrote:
>     Have you changed the label and moved it to first position yet?
> 
> Jan Grulich wrote:
>     Yes, I did that immediately when you told me to do so.

Great!
Then it's good to go from the usability side.


- Thomas


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


On March 9, 2016, 4:15 p.m., Jan Grulich wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127263/
> -----------------------------------------------------------
> 
> (Updated March 9, 2016, 4:15 p.m.)
> 
> 
> Review request for Plasma, Solid, KDE Usability, and Kai Uwe Broulik.
> 
> 
> Repository: powerdevil
> 
> 
> Description
> -------
> 
> This action adds an option to turn off wifi/wwan/bt once you switch profile (e.g you unplugg the power cable and start running on battery).
> 
> One more thing. Due to usage of NetworkManagerQt I had to add add_definitions(-DQT_NO_KEYWORDS) into CMakeLists.txt  like we do in plasma-nm to avoid compilation error (thanks to NetworkManager) and replace all keywords by their Qt equivalent (e.g signals ? Q_SIGNALS).
> 
> How it behaves:
> 1) Switching from "AC" profile to "battery" (or from "battery" to "low battery" which is the same situation):
>    a) When the action is enabled in "AC" profile and options to turn off wifi/wwan/bt are enabled
>       x) Switching to a profile where the action is enabled too and options are turned on ? will do nothing as they should be already turned off
>       y) Switching to a profile where the action is enabled too but options are turned off ? will do nothing as the "battery" profile is more conservative and we have those devices disabled already in less conservative profile
>       z) Switching to a profile where the action is disabled ? will do nothing as there is nothing to do
>    b) When the action is enabled in "AC" profile and options to turn off wifi/wwan/bt are disabled
>       ? this should behave according to the more conservative profile, if the options are enabled then all devices will be disabled too
>    C) When the action is disabled in "AC" profile
>       ? should behave as in the case 1-b
> 2) Switching from "battery" profile to "AC" profile (or from "low battery" to "battery" which is the same situation):
>    a) When the action is enabled in "battery" profile and options to turn off wifi/wwan/bt are enabled
>       x) Switching to a profile where the action is enabled too and options are turned on ? will do nothing as it's same setup
>       y) Switching to a profile where the action is enabled too and options are turned off ? will turn on the wifi/wwan/bt
>       z) Switching to a profile where the action is disabled ? will recover the previous state of the wifi/wwan/bt
>    b) When the action is enabled in "battery" profile and options to turn off wifi/wwan/bt are disabled
>       x) Switching to a profile where the action is enabled too and options are turned on ? will do nothing as the option in "battery" should be ignored in this case due to more conservative profile
>       y) Switching to a profile where the action is enabled too and options are turned off ? will do nothing, same configuration
>       z) Switching to a profile where the action is disabled ? will recover the previous state of the wifi/wwan/bt
>    c) When the action is diabled in "battery" profile
>       ? will just recover the previous state of the wifi/wwan/bt as there is nothing to change according to the new profile
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt e7fff17 
>   daemon/CMakeLists.txt bbfe191 
>   daemon/actions/bundled/CMakeLists.txt 45abea3 
>   daemon/actions/bundled/org.kde.Solid.PowerManagement.Actions.WirelessPowerSaving.xml PRE-CREATION 
>   daemon/actions/bundled/powerdevilwirelesspowersavingaction.desktop PRE-CREATION 
>   daemon/actions/bundled/wirelesspowersaving.h PRE-CREATION 
>   daemon/actions/bundled/wirelesspowersaving.cpp PRE-CREATION 
>   daemon/actions/bundled/wirelesspowersavingconfig.h PRE-CREATION 
>   daemon/actions/bundled/wirelesspowersavingconfig.cpp PRE-CREATION 
>   daemon/powerdevilactionpool.cpp 2091879 
> 
> Diff: https://git.reviewboard.kde.org/r/127263/diff/
> 
> 
> Testing
> -------
> 
> I did some basic testing like (un)plugging the power cable and checking whether it applied the correct configuration.
> 
> 
> Thanks,
> 
> Jan Grulich
> 
>

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