Comment 6 for bug 348377

Revision history for this message
Noel J. Bergman (noeljb) wrote :

David, to be clear, you do know that filtering by service type already exists in the code, right? That is one of the three configuration options that do exist, along with whether or not to provide notifications and whether or not to ignore local services.

In Bug 33517, Mark Shuttleworth suggests that we have the applet use append mode. There is only one call in the code for doing notification:

    def display_notification(self, title, message, iconfile = "file:///usr/share/service-discovery-applet/icons//48x48/service-discovery-applet.png"):
 try:
     if self.show_notifications == True:
                n = pynotify.Notification(title, message, iconfile)
# x,y = self.applet.window.get_origin()
# n.set_hint("x",x)
# n.set_hint("y",y)
# n.attach_to_widget(self.applet.window)
                n.show()
 except:
     print "can't use notification daemon"
            pass

The hinting code is commented out in both Intrepid and Jaunty, so no change there. If I understand Mark's suggestion, the correct change would be:

# diff -u service-discovery-applet~ service-discovery-applet
--- service-discovery-applet~ 2008-06-12 08:18:07.000000000 +0200
+++ service-discovery-applet 2009-03-26 03:06:43.000000000 +0100
@@ -249,6 +249,7 @@
 # n.set_hint("x",x)
 # n.set_hint("y",y)
 # n.attach_to_widget(self.applet.window)
+ n.set_hint("x-canonical-append", "allowed")
                 n.show()
  except:
      print "can't use notification daemon"

I've made the change locally, and will try it out.