Bug 302111

Summary:	battery's brightness doesn't sync with the one from the settings.
Product:	[Plasma] plasma4	Reporter:	Giorgos Tsiapaliokas <giorgos.tsiapaliokas>
Component:	widget-battery	Assignee:	Lukáš Tinkl <lukas>
Status:	RESOLVED FIXED
Severity:	normal	CC:	antonis.tsiapaliokas, giorgos.tsiapaliokas, lukas, viranch.mehta
Priority:	NOR
Version:	4.9-git
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:	http://commits.kde.org/kde-workspace/1d08f4779a26ebba757111fd99182034e5282624	Version Fixed In:	4.9.4
Attachments:	brightness doesn't sync brightness doesn't change if the power state changes

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