[Kde-hardware-devel] Review Request 121508: Allow turning off keyboard in dim display action

[Kai Uwe Broulik]

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

Review request for Solid.


Repository: powerdevil


Description
-------

One thing that always bugged me is that my keyboard backlight stays on even when the screen got turned off/dimmed by power management. This patch adds a checkbox (enabled by default) that turns off the keyboard as soon as it starts dimming the display.
It does not gradually taper down the brightness since keyboard backlights usually aren't that finegrained anyway.


Diffs
-----

  daemon/actions/bundled/dimdisplay.h 1564df1 
  daemon/actions/bundled/dimdisplay.cpp e1bb4f0 
  daemon/actions/bundled/dimdisplayconfig.h acf5071 
  daemon/actions/bundled/dimdisplayconfig.cpp 2fe67dc 

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


Testing
-------

Works as expected. Waited a bit for it to start dimming the display and keyboard turned off. After moving my mouse it was restored to its previous value.

I didn't find a way to conditionally disable the checkbox in case no keyboard backlight was present. But I didn't want to add yet another separate action (so I could use the isSupported() thing).


File Attachments
----------------

Settings in action
  https://git.reviewboard.kde.org/media/uploaded/files/2014/12/14/f4650b04-42c1-4d4b-b772-710623300fa9__dimkeyboard.png


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20141214/8198b32b/attachment.html>