Bug 341873

Summary: Brightness controls don't work when PM is disabled
Product: [Plasma] Powerdevil Reporter: Kai Uwe Broulik <kde>
Component: generalAssignee: Plasma Development Mailing List <plasma-devel>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: http://commits.kde.org/powerdevil/f2f4487e626e357a036e5c64f28643d621dc0573 Version Fixed In:
Sentry Crash Report:

Description Kai Uwe Broulik 2014-12-14 12:20:00 UTC 
When unchecking the "Enable power management" checkbox in battery monitor, changing brightness (both screen and keyboard) don't work until you check it again. It's not a bug in the battery monitor since even calling PowerDevil's brightness interface via dbus directly doesn't work either.

Reproducible: Always

Steps to Reproduce:
1. Open battery monitor
2. Uncheck "Enable power management"
3. Try to change screen or keyboard brightness

Actual Results:  
Nothing happens

Expected Results:  
Still works even with it unchecked
Comment 1 Kai Uwe Broulik 2014-12-14 12:34:48 UTC 
Git commit f2f4487e626e357a036e5c64f28643d621dc0573 by Kai Uwe Broulik.
Committed on 14/12/2014 at 12:32.
Pushed by broulik into branch 'master'.

Revert "Don't show OSD when setting brightness absolutely"

This breaks changing the brightness when PM is disabled since then only "explicit" user actions
are allowed. Now the OSD is shown again when changing the brightness but functional brightness
controls are more important. This should be fixed differently later.

This reverts commit 612b0178934ea9ba0482013a0f09c3f596f46d13.

M  +2    -0    daemon/actions/bundled/brightnesscontrol.cpp
M  +2    -0    daemon/actions/bundled/keyboardbrightnesscontrol.cpp

http://commits.kde.org/powerdevil/f2f4487e626e357a036e5c64f28643d621dc0573