After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 658493 - emit a change signal, when user changes screen brightness
emit a change signal, when user changes screen brightness
Status: RESOLVED FIXED
Product: gnome-settings-daemon
Classification: Core
Component: power
3.1.x
Other Linux
: Normal normal
: ---
Assigned To: Richard Hughes
gnome-settings-daemon-maint
Depends on:
Blocks:
 
 
Reported: 2011-09-07 17:29 UTC by Ritesh Khadgaray ( irc:ritz)
Modified: 2012-02-22 12:36 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
emit change signal, when brightness is changed from userspace (3.07 KB, patch)
2011-09-07 17:29 UTC, Ritesh Khadgaray ( irc:ritz) 		none Details | Review

Description Ritesh Khadgaray ( irc:ritz) 2011-09-07 17:29:33 UTC 
Created attachment 195913 [details] [review]
emit change signal, when brightness is changed from userspace

emit a change signal, when user changes screen brightness.

use case:

  this would allow one to manage screen brightness from gnome shell, without polling for change.
Comment 1 Ritesh Khadgaray ( irc:ritz) 2011-09-07 17:30:06 UTC 
gnome shell extension to test with - http://people.redhat.com/rkhadgar/personal/gnome3-tablet/brightness.tar.xz
Comment 2 Richard Hughes 2011-09-07 17:41:49 UTC 
Makes sense to me, patch looks fine too. hadess, what do you think?
Comment 3 Bastien Nocera 2011-09-07 17:50:19 UTC 
Wouldn't it make sense to just show those as properties, so that you have all the information to present it right there, and notifications for free?

FWIW, that's a 3.4 item, not sure it's the right time to tweak the D-Bus API.
Comment 4 Andrew Cowie 2012-02-22 10:27:09 UTC 
Can we get this into 3.4? The brightness extension would be useful.

AfC
Comment 5 Richard Hughes 2012-02-22 12:36:54 UTC 
I've pushed the patch for 3.4

commit 0e71d83c56108691c152c4c4be88abf6fe3ced2d
Author: Ritesh Khadgaray <khadgaray@gmail.com>
Date:   Wed Feb 22 12:33:42 2012 +0000

    power: Emit a Changed() signal when the backlight changes
    
    Signed-off-by: Richard Hughes <richard@hughsie.com>

We don't want to do this as a property, as reading back the new value is semi-expensive and if nothing cares then we don't want to do this every time.