-----------------------------------------------------------
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

Reply via email to