> On Nov. 23, 2011, 8:18 a.m., Richard Nelson wrote:
> > Quick comment regarding behavior...we really should not play any audio 
> > during the teleport blackout as it is disconcerting to feel like you are 
> > still in an area (hearing the audio) but not be able to see or interact 
> > with anything.  Everything else sounds great.
> 
> Jonathan Yap wrote:
>     The reason I changed the behavior when the teleport progress bar was 
> showing was to keep the state engine logic simple -- it activates when there 
> is change in your current parcel music URL.
>     
>     Otherwise you have to deal with various cases as exceptions to how the 
> state engine processes stream transitions.
>     
>     If you force-fade the current stream when the teleport progress bar 
> appears and the teleport fails you'll have to fade the stream back in.  I am 
> not sure if there are other edge cases that might appear.
>     
>     I suggest trying Oz's PO test viewer (there is a download link in a jira 
> comment).  Personally I felt having the music keep playing and then fade out 
> and in when you arrive was more enjoyable (you can consider it in-flight 
> entertainment).  This also might tie in well with any improvements to the 
> progress bar screen no longer being a blank page as was discussed in one of 
> Oz's user group meetings.
>
> 
> Oz Linden wrote:
>     On the other hand, testing in the current Development viewer [Second Life 
> 3.2.4 (245662)], when I started from a parcel with a stream and TP'ed to a 
> new region&parcel without one, I got a sudden cutoff when the TP screen came 
> up but the stream restarted when the screen dropped and then cut off quickly 
> (presumably because new parcel properties had arrived).  That's arguably not 
> as good as a fade after teleport.

One of the in-progress changes to the teleport/log in screen would involve 
playing video+audio on that login screen, which should not be fighting with 
background audio.


- Richard


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/520/#review1090
-----------------------------------------------------------


On Nov. 23, 2011, 5:59 a.m., Jonathan Yap wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/520/
> -----------------------------------------------------------
> 
> (Updated Nov. 23, 2011, 5:59 a.m.)
> 
> 
> Review request for Viewer.
> 
> 
> Summary
> -------
> 
> Audio fading in has been added for when a music stream starts.  Audio fading 
> out has been added for when there is a change in the music stream that is 
> playing.
> 
> Two dead files have been eliminated.
> 
> An existing bug in how music was not being paused correctly has been fixed.
> 
> When you are teleporting you will hear the music stream from the place you 
> are leaving.  This is a change in behavior; previously the music stream was 
> stopped when the teleport progress bar was being displayed.
> 
> This code change affects several areas where music is started or stopped.  
> The new code has evolved significantly, so please look for things that might 
> not "make sense."
> 
> 
> This addresses bug STORM-591.
>     http://jira.secondlife.com/browse/STORM-591
> 
> 
> Diffs
> -----
> 
>   doc/contributions.txt 8b455c1b7a5e 
>   indra/newview/lloverlaybar.h 8b455c1b7a5e 
>   indra/newview/lloverlaybar.cpp 8b455c1b7a5e 
>   indra/newview/llpanelnearbymedia.h 8b455c1b7a5e 
>   indra/newview/llpanelnearbymedia.cpp 8b455c1b7a5e 
>   indra/newview/llvieweraudio.h 8b455c1b7a5e 
>   indra/newview/llvieweraudio.cpp 8b455c1b7a5e 
>   indra/newview/llviewermedia.cpp 8b455c1b7a5e 
>   indra/newview/llviewerparcelmgr.cpp 8b455c1b7a5e 
> 
> Diff: http://codereview.secondlife.com/r/520/diff
> 
> 
> Testing
> -------
> 
> See the massive test plan in the jira.
> 
> 
> Thanks,
> 
> Jonathan
> 
>

_______________________________________________
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