Bug 182400

Summary:	Visual notification of screen brightness change
Product:	[Frameworks and Libraries] solid	Reporter:	Ruchir Brahmbhatt <ruchir.brahmbhatt>
Component:	powermanagement-daemon	Assignee:	Dario Freddi <drf>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	nathanel.titane, niklas.laxstrom
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	openSUSE
OS:	Unspecified
Latest Commit:		Version Fixed In:

Description Ruchir Brahmbhatt 2009-01-30 11:35:50 UTC

Version:            (using KDE 4.2.0)
Installed from:    SuSE RPMs

There should be OSD notification when we change screen brightness in laptop.

Comment 1 Niklas Laxström 2010-04-15 10:45:30 UTC

Works for me in KDE 4.4.2 when powerdevil is enabled.

Comment 2 Ruchir Brahmbhatt 2010-04-15 10:53:39 UTC

(In reply to comment #1)
> Works for me in KDE 4.4.2 when powerdevil is enabled.

I'm also on 4.4.2 but there is no OSD notification of brightness change/level. Do you get any OSD notification like the way you get for volume change?

Comment 3 Niklas Laxström 2010-04-15 10:58:17 UTC

(In reply to comment #2)
To clarify:
* I see OSD-style scrollbar when changing brightness manually from keyboard. It looks different from the volume OSD.
* I get no OSD for automatic changes in brightness, nor if I change it using other methods than from my keyboard

Comment 4 Ruchir Brahmbhatt 2010-04-15 11:28:49 UTC

Strange, I don't see OSD notification of screen brightness change from laptop keyboard.
Qt: 4.6.2
KDE Development Platform: 4.4.2 (KDE 4.4.2) "release 234"

Comment 5 Felix Geyer 2010-05-05 22:52:23 UTC

*** Bug 221479 has been marked as a duplicate of this bug. ***

Comment 6 Felix Geyer 2010-05-09 23:17:42 UTC

SVN commit 1124692 by fgeyer:

Show an OSD notification when the laptop brightness keys are pressed and
change the brightness if it's not handled by the hardware or the kernel.

Quick implementation overview:
The PowerDevil daemon registers global shortcuts for the
Qt::Key_MonBrightnessUp/Down keys and forwards them to Solid.
Solid then checks if it needs to adjust the brightness.
After that PowerDevil emits the dbus signal brightnessChanged().
The battery applet listens to that signal and displays an OSD
simliar to the one KMix shows.

http://reviewboard.kde.org/r/1942/

BUG: 182400


 M  +8 -0      libs/solid/control/ifaces/powermanager.h  
 M  +6 -0      libs/solid/control/powermanager.cpp  
 M  +16 -0     libs/solid/control/powermanager.h  
 M  +2 -1      plasma/generic/applets/battery/CMakeLists.txt  
 M  +39 -7     plasma/generic/applets/battery/battery.cpp  
 M  +6 -0      plasma/generic/applets/battery/battery.h  
 A             plasma/generic/applets/battery/brightnessosdwidget.cpp   [License: GPL (v2+)]
 A             plasma/generic/applets/battery/brightnessosdwidget.h   [License: GPL (v2+)]
 M  +23 -4     powerdevil/daemon/PowerDevilDaemon.cpp  
 M  +2 -0      powerdevil/daemon/PowerDevilDaemon.h  
 M  +4 -0      powerdevil/daemon/org.kde.PowerDevil.xml  
 M  +54 -7     solid/hal/halpower.cpp  
 M  +4 -0      solid/hal/halpower.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1124692