----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/451/#review998 -----------------------------------------------------------
do not ship until fixing the pointer problem - Richard On Aug. 26, 2011, 8:38 a.m., Vadim ProductEngine wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/451/ > ----------------------------------------------------------- > > (Updated Aug. 26, 2011, 8:38 a.m.) > > > Review request for Viewer and Seth ProductEngine. > > > Summary > ------- > > Reason: Showing the confirmation dialog resets object selection, > thus there is nothing to derez. > > Fix: Save selection until user answers in the confirmation dialog. > > I didn't investigate why the bug occurred only for distant object > (must be some internal LLSelectMgr magic). > > > This addresses bug STORM-638. > http://jira.secondlife.com/browse/STORM-638 > > > Diffs > ----- > > indra/newview/llviewermenu.cpp 64ed6f9362af > > Diff: http://codereview.secondlife.com/r/451/diff > > > Testing > ------- > > See acceptance criteria in the JIRA ticket. > > > Thanks, > > Vadim > >
_______________________________________________ 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
