-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/197/
-----------------------------------------------------------

(Updated July 12, 2011, 11:23 a.m.)


Review request for Viewer.


Summary (updated)
-------

Add optional range ring to the mini-map -- one centered on you with a radius of 
20m to show local chat range.

By default the range ring is off.

To turn it on you right click on the mini-map and pick the menu entry "Range 
Ring".


This addresses bug Storm-1068.
    http://jira.secondlife.com/browse/Storm-1068


Diffs
-----

  doc/contributions.txt aed94e854443 
  indra/newview/app_settings/settings.xml aed94e854443 
  indra/newview/llnetmap.h aed94e854443 
  indra/newview/llnetmap.cpp aed94e854443 
  indra/newview/skins/default/colors.xml aed94e854443 
  indra/newview/skins/default/xui/en/menu_mini_map.xml aed94e854443 

Diff: http://codereview.secondlife.com/r/197/diff


Testing
-------

Tested with another avatar.  When they are outside the ring they cannot see my 
local chat and when they are inside it they can, so the ring's radius is set 
correctly.

Flew over various types of land to make sure my color choice (blue @10%) was 
always visible.

Enabled range ring, logged out and back on; range ring is still present on 
mini-map.

Panned mini-map, range ring remains centered over avatar.

Used mouse wheel to zoom in and out as far as possible.  Size of range ring 
varied with change in zoom level.


Thanks,

Jonathan

_______________________________________________
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

Reply via email to