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 570340 - Misleading comment in brightness_battery GConf key
Misleading comment in brightness_battery GConf key
Status: RESOLVED OBSOLETE
Product: gnome-power-manager
Classification: Deprecated
Component: general
2.24.x
Other Linux
: Normal minor
: ---
Assigned To: GNOME Power Manager Maintainer(s)
GNOME Power Manager Maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2009-02-03 10:31 UTC by Josselin Mouette
Modified: 2012-03-23 09:42 UTC
See Also:
GNOME target: ---
GNOME version: 2.23/2.24


Description Josselin Mouette 2009-02-03 10:31:37 UTC 
Configuring the amount of brightness dimming when on battery is done with the /apps/gnome-power-manager/backlight/brightness_battery GConf key, which comment says:
“The brightness of the display when on battery power. Possible values are between 0 and 100.”

However the code which deals with it is the following:
  gpm_conf_get_uint (backlight->priv->conf, GPM_CONF_BACKLIGHT_BRIGHTNESS_BATT, &value);
  scale = (100 - value) / 100.0f;
  brightness *= scale;

Which means not only this is a multiplicative factor, but that it is inverted. This is actually the percentage of which the brightness is reduced when going on battery.

At the very least, the comment should be updated to indicate that, but there should probably be some more intuitive setting instead – one that increases brightness when the value is increased.