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