[Sugar-devel] [PATCH sugar 1/7] SugarEventIcon: Add a hippo-free implementation of the CanvasIcon

[Benjamin Berg]

Hi,

On Mon, 2012-08-06 at 12:04 +0200, Simon Schampijer wrote:
> Signed-off-by: Simon Schampijer <simon at laptop.org>
> [moved from sugar-toolkit to sugar]
> Signed-off-by: Sascha Silbe <silbe at activitycentral.com>
Reviewed-by: Benjamin Berg <benzea at sugarlabs.org>

> +    def __init__(self, **kwargs):
> +        self._buffer = _IconBuffer()
> +        self._alpha = 1.0
> +
> +        gtk.EventBox.__init__(self)
> +        for key, value in kwargs.iteritems():
> +            self.set_property(key, value)
> +
> +        self._palette_invoker = CursorInvoker()
> +        self._palette_invoker.attach(self)
> +
> +        # HACK to supress the grey background around the icon
> +        # won't be needed in GTK3
> +        self.modify_bg(gtk.STATE_NORMAL, style.COLOR_WHITE.get_gdk_color())

Not really too bad, but I am pretty sure that
gtk.EventBox.set_visible_window(False) should work fine here.