"Battery low" notification displays wrong icon, when Human theme is selected

[Mat Tomaszewski]

Binary package hint: notify-osd

For the Human theme, a new steel icon should be displayed.

[Kenneth Wimer (kwwii)]

the icon exists in the theme as notification-battery-low.svg

[Sebastien Bacher (seb128)]

icon = "notification-battery-low";
...
g_free(icon)

isn't that going to crash?

[Cody Russell (bratsche)]

Eek, you're right. I believe this corrects the problem though.

[Cody Russell (bratsche)]

I forgot to diff with -p1

[Launchpad Janitor (janitor)]

This bug was fixed in the package gnome-power-manager - 2.24.2-2ubuntu8

---------------
gnome-power-manager (2.24.2-2ubuntu8) jaunty; urgency=low

  [ Cody Russell ]
  * debian/patches/04-battery-low-icon-change.patch:
    - use correct icon on the battery low notification (lp: #344886)
  * debian/patches/97_remove-discharging-notify.patch:
    - don't display a notification about battery discharging (lp: #355877)

 -- Sebastien Bacher <email address hidden> Wed, 08 Apr 2009 16:46:06 +0200

[Martin Pitt (pitti)]

I reopen this for karmic. The patch applied to 2.24 does not apply at all to 2.26.1, which I'm currently merging. Can you please recreate this against 2.26.1 (bzr get lp:~ubuntu-desktop/gnome-power/devkit-power/, I'll merge it to the ubuntu branch soon) and also forward this upstream?

I guess there might be some more work on icons required anyway, or introduce some general themeability, since the icons in 2.26 changed quite a lot.

Thank you!