On 2011-01-25 1:05, Twisted Laws wrote:
Added a patch to https://jira.secondlife.com/browse/STORM-643 Embed
Minimap into Nearby list of People Sidebar. I know I never make the
meetings that would probably help determine if this would be included
in the viewer, but it does help make viewer 2 just a little better
than it was in my opinion. Additionally, selecting a user in the
nearby avatar list will highlight the avatar on the minimap. The
patch also corrects an error I don't see in the Jira where the minimap
hovertip is incorrect if double click teleport is enabled.
This does look like an interesting feature.
It really makes it significantly easier to take patches if you:
* Create a review on codereview.secondlife.com, properly linked to
the jira issue (see instructions
<https://wiki.secondlife.com/wiki/Code_Review_Tool#Post_Code_To_Be_Reviewed>)
* Post a public mercurial repository based on viewer-development
that has the change added.
If you do both, it really makes the process of review _much_ Faster,
Easier, and More Fun.
I know that it probably seems like significant extra work, but really
it's pretty simple, and if each contributor does that little bit of
extra work, the much smaller number of us on the receiving end can
handle more of them.
_______________________________________________
Policies and (un)subscribe information available here:
http://wiki.secondlife.com/wiki/OpenSource-Dev
Please read the policies before posting to keep unmoderated posting privileges
