[sugar] [PATCH] add a secondary label and icon to palettes WAS Merging sugar-toolkit changes from tomeu repository

Eben Eliason eben.eliason
Tue Apr 1 14:20:14 EDT 2008


OK, here's a newer version of the patch.  It removes the
menu_after_content property and associated code, puts a DEPRECATED
comment with the Palette constructor for future reference, ensures
that the icon passed gets set to the correct size, and cuts a few
lines of previously commented code that are no longer needed.

- Eben

On Tue, Apr 1, 2008 at 10:53 AM, Tomeu Vizoso <tomeu at tomeuvizoso.net> wrote:
> On Tue, Apr 1, 2008 at 4:30 PM, Marco Pesenti Gritti <mpgritti at gmail.com> wrote:
>  > Padding and spacing are property of the container, not of the icon.
>  >  It's common practice in gtk to have menu and toolbars set the icon
>  >  size, so I don't think that would be confusing.
>
>  So let's ship it!
>
>  Tomeu
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: palette_merge.patch
Type: text/x-patch
Size: 11647 bytes
Desc: not available
Url : http://lists.laptop.org/pipermail/sugar/attachments/20080401/b9f00d51/attachment-0001.bin 



More information about the Sugar-devel mailing list