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 687828 - Don't show obsolete power notifications
Don't show obsolete power notifications
Status: RESOLVED FIXED
Product: gnome-settings-daemon
Classification: Core
Component: power
3.6.x
Other Linux
: Normal normal
: ---
Assigned To: Richard Hughes
gnome-settings-daemon-maint
Depends on: 687829
Blocks:
 
 
Reported: 2012-11-07 09:11 UTC by Allan Day
Modified: 2013-06-11 23:05 UTC
See Also:
GNOME target: ---
GNOME version: 3.7/3.8


Attachments
power: Hide critical battery warning when power is plugged (1.29 KB, patch)
2012-11-29 22:39 UTC, Bastien Nocera
committed Details | Review
screenshot (29.62 KB, image/png)
2013-06-09 22:36 UTC, Allan Day
  Details

Description Allan Day 2012-11-07 09:11:32 UTC
See bug 687658.
Comment 1 Richard Hughes 2012-11-07 09:25:05 UTC
We do use notify_close_if_showing() to hide any non-relevent messages when we show a new message. Can you give an example where we show more that one power notification messages at one time? Thanks.
Comment 2 William Jon McCann 2012-11-12 14:45:15 UTC
Bug 687658 is an example, no?
Comment 3 Matthias Clasen 2012-11-29 19:53:37 UTC
Here is an example:

I just got a 'battery critical' notification and scrambled to plug in. Now my battery is happily charing, but this big, scary, red battery icon is sitting in my message tray. It should be removed as soon as I plug in.
Comment 4 Bastien Nocera 2012-11-29 22:39:00 UTC
Created attachment 230248 [details] [review]
power: Hide critical battery warning when power is plugged
Comment 5 Matthias Clasen 2012-11-29 23:59:32 UTC
Review of attachment 230248 [details] [review]:

::: plugins/power/gsd-power-manager.c
@@ +1885,1 @@
                         g_debug ("fully charged, hiding notifications if any");

Debug message now slightly misleading...
Comment 6 Bastien Nocera 2012-12-03 16:06:51 UTC
Attachment 230248 [details] pushed as 1b2caa7 - power: Hide critical battery warning when power is plugged
Comment 7 Allan Day 2013-06-09 22:36:27 UTC
Created attachment 246371 [details]
screenshot

Just saw this on F19/GNOME 3.8.2 - three low power warnings, all in a row.
Comment 8 Bastien Nocera 2013-06-11 16:33:52 UTC
File a new bug please.
Comment 9 Allan Day 2013-06-11 23:05:28 UTC
Filed as bug 702047 .