Bug 341873 - Brightness controls don't work when PM is disabled
Summary: Brightness controls don't work when PM is disabled
Status: RESOLVED FIXED
Alias: None
Product: Powerdevil
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Development Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-12-14 12:20 UTC by Kai Uwe Broulik
Modified: 2014-12-14 12:34 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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