[sugar] Landing patches about the network devices UI

[Walter Bender]

I agree with your assessment. Note that right now (8.2-756), the mesh
icons don't show up on either the Neighborhood view or the Frame; that
is clearly not a satisfactory situation.

regards.

-walter

On Mon, Aug 25, 2008 at 8:19 AM, Marco Pesenti Gritti
<mpgritti at gmail.com> wrote:
> Hello,
>
> we have a couple of patches that should be ready to be reviewed by
> tomorrow, which solves several issues with the UI of network devices.
>
> #6944 UI confuses which AP you are connected to
> #3993 The color of network icon in Home view becomes white after
> restarting Sugar.
> #2866 Network Manager GUI doesn't report success or failure
> #6995 Add a mesh device to the frame and remove mesh devices from
> Neighborhood view
>
> The tickets are a little confusing, so let me summarize what the patches does:
>
> 1 Adds "IP address" to the mesh & wireless palettes, with associated
> changes to their model classes.
> 2 Removes the "Disconnect" or "Turn On/off" entries from the
> wireless/mesh palettes.
> 3 Makes both frame icons pulse.
> 4 Don't show the mesh icons in the mesh view, instead show them in the frame.
> 5 Fix some iconsistency in the icon states by cleaning up the code.
>
> mtd did quite a bit of testing on them already, but they are pretty
> invasive and there is some risk of regressions.
>
> My opinion is:
>
> 2 is controversial and should be left as is for 8.2
> 1, 5 are important and we should try to get them in.
> 4 would be nice to have but I don't consider it essential.
> 3 should be delayed unless it's small and it's easier to take it then
> to refactor patches.
>
> 1,3,5 has been submitted for review as patch A. 2, 4 will be submitted
> today as patch B. My suggestion would be:
>
> * Rip off 3 from patch A if it's worth it and land it for 8.2.0 (before Friday)
> * Do *not* land patch B for 8.2.0
>
> mtd has some free time today, so if we can let him know what we want
> and don't want to land soon it would be great.
>
> Thanks,
> Marco
> _______________________________________________
> Sugar mailing list
> Sugar at lists.laptop.org
> http://lists.laptop.org/listinfo/sugar
>