Bug 321623 - Brightness sliders do not update when changing backlight with keys
Summary: Brightness sliders do not update when changing backlight with keys
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: widget-battery (show other bugs)
Version: 4.10.80
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-06-26 08:28 UTC by Martin Klapetek
Modified: 2013-06-26 09:44 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Klapetek 2013-06-26 08:28:58 UTC
Macbook has keys for changing keyboard backlight. If I use keys to change the backlight, the slider in battery popup does not change but stays at the last set position.

Steps to reproduce:
1. Move the slider to 50%
2. Set the backlight down to 0% using keys
3. Check the widget, slider still at 50%
Comment 1 Kai Uwe Broulik 2013-06-26 08:55:35 UTC
Funny, I just discovered this behavior myself.
From what I know that was done on purpose to prevent races between the backend and the battery monitor which caused trouble in the past, eg. the battery monitor constantly setting a different brightness.
I already have an idea for that though. The problem definitly lies in the battery monitor as the powermanagement dataengine is updated properly (can be easily seen with plasmaengineexplorer, there the value is correct).

It's true for both brightness types (keyboard and backlight).
Comment 2 Kai Uwe Broulik 2013-06-26 09:44:50 UTC
Git commit 47e294c560dbef923cd86359dd23cd40942b0724 by Kai Uwe Broulik.
Committed on 26/06/2013 at 09:43.
Pushed by broulik into branch 'master'.

Fix screen brightness not being updated properly.
I moved the updateBrightness function to the logic.js but forgot that the pmSource
will not be available there. Now passing pmSource to the function. Works.

M  +6    -6    plasma/generic/applets/batterymonitor/contents/code/logic.js
M  +1    -1    plasma/generic/applets/batterymonitor/contents/ui/batterymonitor.qml

http://commits.kde.org/kde-workspace/47e294c560dbef923cd86359dd23cd40942b0724