Bug 302111 - battery's brightness doesn't sync with the one from the settings.
Summary: battery's brightness doesn't sync with the one from the settings.
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: widget-battery (show other bugs)
Version: 4.9-git
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Lukáš Tinkl
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-06-18 10:10 UTC by Giorgos Tsiapaliokas
Modified: 2012-11-21 19:58 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 4.9.4


Attachments
brightness doesn't sync (880.45 KB, image/png)
2012-06-18 10:11 UTC, Giorgos Tsiapaliokas
Details
brightness doesn't change if the power state changes (826.48 KB, image/png)
2012-06-18 10:26 UTC, Giorgos Tsiapaliokas
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Giorgos Tsiapaliokas 2012-06-18 10:10:39 UTC
Hello,

how to reproduce,

1.open the settings of the battery plasmoid change the brightness.
2. click ok
3. see if this applies to plasmoid's brightness (it doesn't)

also I have attached a screenshot

Reproducible: Always
Comment 1 Giorgos Tsiapaliokas 2012-06-18 10:11:09 UTC
Created attachment 71904 [details]
brightness doesn't sync
Comment 2 Giorgos Tsiapaliokas 2012-06-18 10:25:32 UTC
Also due to the fact that they don't sync
if I have set the brightness in (zero/the lowest value) during the
on battery state the brightness won't change. ( i have also attached a screenshot)
Comment 3 Giorgos Tsiapaliokas 2012-06-18 10:26:37 UTC
Created attachment 71906 [details]
brightness doesn't change if the power state changes

In my laptop I have full brightness, but I shouldn't have!
Comment 4 Myriam Schweingruber 2012-06-20 06:54:33 UTC
And which version is this about? Please always specify the version you are using and set the version field in your bug reports.
Comment 5 Antonis Tsiapaliokas 2012-06-21 22:08:07 UTC
(In reply to comment #4)
> And which version is this about? Please always specify the version you are
> using and set the version field in your bug reports.

He is using the master branch... In the version field the entry "4.9-git" is the choosen one :)
Comment 6 Myriam Schweingruber 2012-06-22 14:55:42 UTC
(In reply to comment #5)
> (In reply to comment #4)
> > And which version is this about? Please always specify the version you are
> > using and set the version field in your bug reports.
> 
> He is using the master branch... In the version field the entry "4.9-git" is
> the choosen one :)

It now is, it wasn't when I check and asked the question :)
Comment 7 Lukáš Tinkl 2012-07-10 09:28:01 UTC
Looking into it
Comment 8 Gregor Tätzner 2012-11-21 19:58:23 UTC
Git commit 285553edce61f65a88d9067136bf138e44efb149 by Gregor Tätzner.
Committed on 21/11/2012 at 20:47.
Pushed by gregort into branch 'KDE/4.9'.

Fix screenBrightnessChanged signal in upower backend

emit onBrightnessChanged every time when a new brightness gets set and
not just on brightnessKey press
Related: bug 302160
FIXED-IN: 4.9.4
REVIEW: 107398

M  +12   -8    powerdevil/daemon/backends/upower/powerdevilupowerbackend.cpp

http://commits.kde.org/kde-workspace/285553edce61f65a88d9067136bf138e44efb149
Comment 9 Gregor Tätzner 2012-11-21 19:58:26 UTC
Git commit 1d08f4779a26ebba757111fd99182034e5282624 by Gregor Tätzner.
Committed on 21/11/2012 at 20:47.
Pushed by gregort into branch 'master'.

Fix screenBrightnessChanged signal in upower backend

emit onBrightnessChanged every time when a new brightness gets set and
not just on brightnessKey press
Related: bug 302160
FIXED-IN: 4.9.4
REVIEW: 107398

M  +12   -8    powerdevil/daemon/backends/upower/powerdevilupowerbackend.cpp

http://commits.kde.org/kde-workspace/1d08f4779a26ebba757111fd99182034e5282624