Looks good to me. Though:

+        Palette.__init__(self, None, None, primary_text=activity_info.name,
+                         icon=activity_icon)

Do we really need the two None? I hoped we could just make those
default to None in sugar.graphics.Palette...

Marco
_______________________________________________
Sugar mailing list
Sugar@lists.laptop.org
http://lists.laptop.org/listinfo/sugar

Reply via email to