[opensource-dev] STORM-643

Oz Linden (Scott Lawrence) oz at lindenlab.com
Tue Jan 25 09:46:53 PST 2011


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.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110125/05e2eed1/attachment.htm 


More information about the opensource-dev mailing list