[sugar] [PATCH] (partial) AP palette patch

Simon Schampijer simon
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-devel mailing list