----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/552/#review1168 -----------------------------------------------------------
Ship it! Ship It! - Oz Linden On Feb. 18, 2012, 5:24 a.m., Jonathan Yap wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/552/ > ----------------------------------------------------------- > > (Updated Feb. 18, 2012, 5:24 a.m.) > > > Review request for Viewer. > > > Description > ------- > > Part 1 > Open your inventory's Animations folder > Double click on an animation > Click on Play Inworld: Play Inworld is replaced by a Stop button > > Observed behavior: It is still possible to click on Play Locally, which also > is replaced by a Stop button. Clicking on either Stop works and resets the > floater back to having both Play buttons showing. > > Expected behavior: When one of the play buttons is clicked the other should > be disabled. > > > Part 2 > 1 .To reproduce, get any non looping animation. ie, one that plays once and > then stops. > 2. Right click in inventory and select play in world. > 3. The animation dialog appears with play locally and play in world buttons. > 4. When the animation is playing, the play in world button changes to a stop > button, which makes it stop playing. > Now here's the problem: > 5. When the animation finishes of it's own accord, the button still says > stop. To play it again, you have to click the redundant stop button to make > it change back to Play in World, then click that to play again. > > Expected behaviour, is that when an animation finishes, the stop button > should change back of it's own accord. > > > This addresses bug STORM-1807. > http://jira.secondlife.com/browse/STORM-1807 > > > Diffs > ----- > > doc/contributions.txt 0a41a8750048 > indra/newview/llinventorybridge.cpp 0a41a8750048 > indra/newview/llpreviewanim.h 0a41a8750048 > indra/newview/llpreviewanim.cpp 0a41a8750048 > indra/newview/skins/default/xui/en/floater_preview_animation.xml > 0a41a8750048 > > Diff: http://codereview.secondlife.com/r/552/diff/diff > > > Testing > ------- > > See test plan in jira. > > > 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