Bug 341873

Summary:	Brightness controls don't work when PM is disabled
Product:	[Plasma] Powerdevil	Reporter:	Kai Uwe Broulik <kde>
Component:	general	Assignee:	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:

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