> 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