[sugar] [PATCH] (partial) AP palette patch
Simon Schampijer
simon at schampijer.de
Wed Apr 9 18:47:36 EDT 2008
Hi Eben,
I made the following changes to the patch:
- display the AP icon in the palette in color
- display the badge as well
- removed the channel info in the secondary-text (for now)
What do you mean here?
+ self.props.icon_name = icon_name
+ # This breaks style guidelines; we should store a reference
+ self._palette._icon.props.icon_name = icon_name
The Connect icon is 'dialog-ok' the disconnect 'media-eject', should we
make this 'dialog-cancel' to be consistent?
Another thought that come to my mind - the AP icon in the frame should
maybe come with a badge as well.
Best,
Simon
Eben Eliason wrote:
> This is my attempt at cleaning up the appearance and behavior of the
> palettes attached to APs in the neighborhood view. I can't see APs in
> jhbuild, so I can't test it. It compiles, but I won't promise it
> runs. Also, there are a couple of bits of info that I want to show
> that I didn't know how to obtain (I didn't look very hard), as
> indicated in the comments. In other words, this patch is a starting
> point, and I'd appreciate it if someone with the know-how and ability
> to test it would finish it up! Thanks.
>
> - Eben
>
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> Sugar mailing list
> Sugar at lists.laptop.org
> http://lists.laptop.org/listinfo/sugar
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ap_mesh.patch
Type: text/x-patch
Size: 4851 bytes
Desc: not available
Url : http://lists.laptop.org/pipermail/sugar/attachments/20080410/fb121320/attachment.bin
More information about the Sugar
mailing list