This text on the battery tab is kind of weird, I'm not sure I understand the signification. When I hover the checkbox, a tooltip says me it's not supported on my platform (and I can't select it, anyway, it's greyed out), but I don't know what it is. Maybe it'd be a good idea to show a little help text?
I didn't understand exactly what you are talking about, is the checkbox for the power save in the battery tab ?
(In reply to comment #1) > I didn't understand exactly what you are talking about, is the checkbox for the > power save in the battery tab ? Yes, first checkbox, between “When battery charge is critical, do…“ and “Enable battery state notification“
(In reply to comment #2) > Yes, first checkbox, between “When battery charge is critical, do…“ and “Enable > battery state notification“ I see, actually the power manager test the method SetPowerSave using the current power_management.is_powersave_set value to check if we can send such event when running on battery if not we just disable it, i'll try to be more clear in this checkbox.
(In reply to comment #3) > (In reply to comment #2) > > Yes, first checkbox, between “When battery charge is critical, do…“ and “Enable > > battery state notification“ > > I see, actually the power manager test the method SetPowerSave using the > current > power_management.is_powersave_set value to check if we can send such event when > running on battery if not we just disable it, i'll try to be more clear in this > checkbox. Hmhm, but I don't even know what it is :)
SetPowerSave using the current power_management.is_powersave_set
Yes but *what* is SetPowersave or power_management.is_powersave_set I don't know, and I would assume the regular user won't either. So it may be a good idea to show a little help on this.
(In reply to comment #6) > Yes but *what* is SetPowersave or power_management.is_powersave_set > > I don't know, and I would assume the regular user won't either. So it may be a > good idea to show a little help on this. Not sure if i'll keep this option, since we are managing cpu, dpms and screen luminosity, so what hal setpowersave methods can do more!
(In reply to comment #7) > Not sure if i'll keep this option, since we are managing cpu, dpms and screen > luminosity, so what hal setpowersave methods can do more! I don't get it. What this option supposed to do, in the end? Is it a global stuff to put all components (cpu/dpms/screen lum) in a low-power state?
(In reply to comment #8) > I don't get it. What this option supposed to do, in the end? Is it a global > stuff to put all components (cpu/dpms/screen lum) in a low-power state? No it's not a global option, it's a method in hal see org.freedesktop.Hal.Device.SystemPowerManagement.method_names. It depends on the script hal-system-power-set-power-save-linux wish uses a util to do a full powersave like pm-powersave. but i'm not sure if i'll keep this option.
Fixed in svn. we don't show this check box at all if no powersave method is supported by HAL.
Closing old bugs to keep things organized.