[Sugar-devel] [PATCH] Draw accelerator in Palette SL #3459

[Daniel Drake]

On Wed, Apr 18, 2012 at 2:35 PM, Simon Schampijer <simon at schampijer.de> wrote:
> The accelerator in the primary information in the Palette
> has not been drawn because there was not enough space
> reserved for it. The preferred size we get back for the
> Palette window does not include the accelerator of the
> Gtk.AccelLabel. We need to include that in our calculation for
> the Palette size.
>
> In order to make that information available which is part
> of the Palette class we need to pass the instance to the
> PaletteWindowWidget instance.

Looks good. Having the do_get_preferred_width implementation in the
Palette class was a definite oddity because Palette is not a GTK
widget and so this was never called.

Thanks
Daniel