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


Changes
-------

Fix possible compiling issue on Linux


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 (updated)
-----

  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