[sugar] Landing patches about the network devices UI

Kimberley Quirk kim
Mon Aug 25 11:34:48 EDT 2008


Just spoke with Ricardo and he stated that the wireless firmware we  
added to this build was for test purposes only and not the firmware  
that we want to ship. It has other test code in it and the simple mesh  
feature was specifically turned off; so we can't use this firmware for  
system testing and we need to revert until we have a better firmware  
release with the bug fix on top of what was otherwise working well.

The issue I pointed out to him (and he will work with Marvell) is that  
we are only a few weeks away from our desired release date and the  
blocking bug in the firmware needs to be addressed in the next day or  
two. We really need a version of wireless firmware that we can be  
using in large scale testing by the end of the week.

Thanks for your help in driving this, Ricardo!

Kim



On Aug 25, 2008, at 9:13 AM, Walter Bender wrote:

> 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
>>




More information about the Sugar-devel mailing list