Low power warning dialog shows unusable options (suspend)

Bug #20091 reported by RichardKinder
10
Affects Status Importance Assigned to Milestone
gnome-applets (Ubuntu)
Fix Released
Low
Daniel Holbach

Bug Description

Minor issue - when the low battery warning dialog comes up, it lists 'suspend
your laptop to save power' when my laptop doesn't support suspend (T40). The
Gnome logout dialog correctly doesn't list suspend as an option (hibernate is
available however). The dialog should only suggest suspension if it's supported
by the laptop.

Screenshot attached.

http://bugzilla.gnome.org/show_bug.cgi?id=314812: http://bugzilla.gnome.org/show_bug.cgi?id=314812

Revision history for this message
RichardKinder (richard-kinder) wrote :

Created an attachment (id=3303)
Screenshot of the offending dialog

The dialog shouldn't give the option of suspending the laptop when suspend to
ram isn't supported by the laptop.

Revision history for this message
Allison Karlitskaya (desrt) wrote :

Will address in 2.14 if we can.

Revision history for this message
Allison Karlitskaya (desrt) wrote :

Created an attachment (id=5170)
patch to enable PMI support in battstat

i added support to detect through PMI if suspend is supported. it's off by
default in upstream since most distributions don't have PMI. here's a patch to
enable it (just #define HAVE_PMI basically).

Revision history for this message
Daniel Holbach (dholbach) wrote :

HAVE_PMI is not in a gnome-applets release yet (2.12.2 was the latest one), but
I'll enable it as soon as we have it.

Revision history for this message
Allison Karlitskaya (desrt) wrote :

eek. forgot to mention that it's only in CVS HEAD so far :)

Revision history for this message
Daniel Holbach (dholbach) wrote :

Release! Release! :-)

Revision history for this message
Daniel Holbach (dholbach) wrote :

 gnome-applets (2.13.1-0ubuntu1) dapper; urgency=low
 .
   * New upstream release "Codename Mobius":
     - This is the first development release of GNOME Applets for the
       GNOME 2.13 series.
     - Split GWeather into the GWeather frontend and libgweather which has the
       parsers for weather servers and the preferences. (Philip Langdale)
     - Code cleanup, gcc 2.95 fixes, etc. (Jens Granseuer, Kjartan Marass,
       Laszlo Peter, Aurelien Jarno)
     - Battery applet text will rotate on side panels (Ryan Lortie)
     - Implement a is_suspend_unavailable() hook for vendors to patch
       (Ryan Lortie) (Ubuntu: #13851)
     - Stop drivemount icons exloderizing the applet (Michael Hofmann)
     - GSwitchit plugin updates (Sergey Udaltsov)
     - French documentation (Christophe Bliard)
     - Locations Love (Ryan Lortie, Farzaneh Sarafraz)
     - Reduce flicker when positioning the mixer (Ronald S. Bultje)
     - Other mixer fixes (Kazuki Shimura, Ronald S. Bultje)
     - Fix a major stickynotes crasher (Jaap A. Haitsma)
     - Translations: zh_TW (Abel Cheung), cs (Miloslav Trmac), gu (Ankit
       Patel), th (Theppitak Karoonboonyanan), es (Francisco Javier F.
       Serrador), gl (Ignacio Casal Quinteiro), en_CA (Adam Weinberger),
       bg (Alexander Shopov), pt_BR (Guilherme de S. Pastore), bn (Mahay
       Alam Khan), nl (Tino Meinen), fa (Masoud Ahmadzadeh, Meelad Zakaria),
       ku (Erdal Ronahi)
   * debian/patches/09_gweather_translation.patch:
     - dropped, done upstream.
   * debian/patches/14_battstat_enable_pmi.patch:
     - patch from Ubuntu #13851 by Ryan Lortie <email address hidden>.
   * debian/gnome-applets-dev:
     - added libgweather/gweather.pc
     - added /usr/lib/*{so,la,a}
   * debian/gnome-applets:
     - added /usr/lib/*.so.*

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.