----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/553/#review1165 -----------------------------------------------------------
Ship it! Ship It! - Oz Linden On Feb. 19, 2012, 1:22 p.m., Jonathan Yap wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/553/ > ----------------------------------------------------------- > > (Updated Feb. 19, 2012, 1:22 p.m.) > > > Review request for Viewer. > > > Description > ------- > > When standing in a parcel it would be helpful to have a graphical indication > in the UI if you are able to build/rez. > > The icon in the (mini) location bar shows parcel properties, not what is > available to you as parcel owner or as a member of a group that has rezzing > enabled as one of your group roles. > > Suggested solution: Gray out the build button if it is present on the > toolbar, just like the Speak button is grayed out if voice is not available. > > > This addresses bug STORM-1808. > http://jira.secondlife.com/browse/STORM-1808 > > > Diffs > ----- > > doc/contributions.txt 0a41a8750048 > indra/newview/app_settings/commands.xml 0a41a8750048 > indra/newview/llagent.cpp 0a41a8750048 > indra/newview/lltoolmgr.h 0a41a8750048 > indra/newview/lltoolmgr.cpp 0a41a8750048 > > Diff: http://codereview.secondlife.com/r/553/diff/diff > > > Testing > ------- > > See test plan in jira. > > I also searched the xml files to be sure no other calls to > LLAgent::isActionAllowed were being made. > > > Thanks, > > Jonathan Yap > >
_______________________________________________ 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
