Re: applet: clear notification actions before adding new ones



On Mon, 2015-01-26 at 17:51 -0500, Mathieu Trudel-Lapierre wrote:
Make sure we only ever have one of the "Don't show this again" button that
will trigger disabling notifications. Otherwise we end up with multiple
instances of the same button in notifications because one gets added every
time a new notification stacks on top of an existing one.

Thanks!  Pushed to 0.9.10, 1.0, and git master.

Dan

---
 src/applet.c |    1 +
 1 file changed, 1 insertion(+)

Index: b/src/applet.c
===================================================================
--- a/src/applet.c
+++ b/src/applet.c
@@ -924,6 +924,7 @@ applet_do_notify (NMApplet *applet,
      notify_notification_set_timeout (notify, NOTIFY_EXPIRES_DEFAULT);
 
      if (applet_notify_server_has_actions () && action1) {
+             notify_notification_clear_actions (notify);
              notify_notification_add_action (notify, action1, action1_label,
                                              action1_cb, action1_user_data, NULL);
      }
_______________________________________________
networkmanager-list mailing list
networkmanager-list gnome org
https://mail.gnome.org/mailman/listinfo/networkmanager-list




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]