Indicate connected AP in Neighborhood view.

[Eben]

Many testers of the latest design have been confused by the lack or indication of the connected AP in the neighborhood. While the new design adds the network node to the Frame, accessible from anywhere, the old design had a white stroke on the connected AP.

While the white stroke was not a good solution, both because it strips the "identity" of the AP and because it just wasn't that noticeable, it's a good idea to have a differentiator here.

[mtd]

cscott has patches:

http://dev.laptop.org/git?p=users/cscott/sugar;a=commitdiff;h=3490dca13798b9d4bf6104ab86f29d7a7c158ff6

http://dev.laptop.org/git?p=users/cscott/sugar;a=commitdiff;h=3d0e39abdf00e859113b8489d7ad865d0c95e371

...so I'm recording his r? herewith.

[mtd]

r+ -- looks cool.

But there are some artwork patches I think we need too (for the icons) so I'll wait to push to git until those are available, though I could just be being jumpy here and anyone more informed is welcome to act on the r+ (of course).

[cscott]

patched rpms for sugar and sugar-artwork at http://dev.laptop.org/~cscott/fer-eben/ now.

I need someone to actually test these, though: I forgot to bring an XO home with me. =(

Test that the icon changes when you connect to an access point, to an (o) icon, and that both the mesh view and the icon in the frame change appropriately. Check that the icon stays correct as the strength of the network changes, if you can. Finally, since the emblem location was part of the new icons Eben created, double check that the 'favorites star' stays in place and doesn't wander as the icon traverses the 'not connected', 'connecting', and 'connected' states.

[cscott]

Eben's patch should be pulled from master to sucrose-0.82 if it passes testing.

[mstone]

I tested these rpms and was very satisfied with the results.

[mtd]

The fill looks odd on the AP icons in the Neighborhood view...see attached screenshot.

Is anyone else seeing this?

[mtd]

screenshot with cscott's sugar and sugar-artwork rpms showing odd fill in AP icons

[cscott]

Hm, those should have been the *unaffected* icons. I'll take a look.

[cscott]

It seems to be deliberate on Eben's part? I can edit the files to remove the extra fill; Eben, which do you prefer?

[cscott]

from irc:

(10:06:16 AM) C. Scott Ananian: eben: what about the filled center in the new ap icons?
(10:06:21 AM) C. Scott Ananian: eben: was that intentional?
(10:06:34 AM) eben: c_scott: Pentagram and I both like them.  We've been using them in mockups for a while.
(10:06:56 AM) C. Scott Ananian: eben: ok, i guess #8554 is ready to go then.

[cscott]

Ok, committed to master and sucrose-0.82. I'm going to attach the patch which changes the icon shape for mesh icons; we're not using this right now, but if Eben ever decides he wants mesh halos, the patch will be right here.

[erikos]

I see the icon :)

[marco]

Package:

sugar-0.82.8-1.olpc3

Code change:

http://dev.laptop.org/git?p=sugar;a=commit;h=915613f9a217a92624fde90fd5d4d022ddcb01a4

[mstone]

Very nice work.

[cscott]

Please test build 761 resulting from build-repo commit b7c11be9.

[mstone-xmlrpc]

changed by irc user m_stone:

We're tentatively satisfied that this works in 8.2-762; relnotes needed. (and maybe an update to the Help activity?)

[gregorio]

Added to resolved section of the release notes. GS