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 338630 - when change brightness on idle, check the current level first.
when change brightness on idle, check the current level first.
Status: RESOLVED FIXED
Product: gnome-power-manager
Classification: Deprecated
Component: general
2.15.x
Other All
: Normal enhancement
: ---
Assigned To: GNOME Power Manager Maintainer(s)
GNOME Power Manager Maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2006-04-15 17:28 UTC by Paul Ionescu
Modified: 2006-04-15 21:40 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Paul Ionescu 2006-04-15 17:28:07 UTC 
On battery, I have set the brightness to 0%, which works well, but then, after the idle timeout, the lcd is bumped to 30% which is the value for idle. 

So I think that maybe we need to check if the idle brightness is higher than the __current__ LCD brightness (not the level set in policy, because brightness may be manually changed), and if it is, ignore the change. Otherwise we end up wasting power.
Comment 1 Richard Hughes 2006-04-15 19:00:09 UTC 
Yes, agreed.
Comment 2 Richard Hughes 2006-04-15 21:40:05 UTC 
2006-04-15  Richard Hughes  <richard@hughsie.com>
 * src/gpm-brightness.{c|h} (gpm_brightness_level_get): Add this new function so we can get at the current percentage value.
 * src/gpm-manager.c (idle_changed_cb): Check that we do not set a higher brightness when we try to lower the brightness on idle. Should fix #338630.

Please check this fixes the issue. Many thanks.