[sugar] [PATCH] News devices tray

Marco Pesenti Gritti mpgritti at gmail.com
Tue Apr 1 15:10:49 EDT 2008


+        TrayIcon.__init__(self, icon_name=_ICON_NAME,
xo_color=profile.get_color())

80 cols

+            self.icon.props.fill_color = style.COLOR_INACTIVE_FILL.get_svg()
+            self.icon.props.stroke_color =
style.COLOR_INACTIVE_STROKE.get_svg()

You have a bunch of these. I suggest icon = self.icon at the top (it
will also make it fit in 80 cols).

r+ with those nitpicks


More information about the Sugar mailing list