Bugzilla – Bug 4364
Pandora dual-screen text display sync issues
Last modified: 2008-12-15 13:05:03 UTC
I haven't filed enough SN bugs lately. :) When I'm listening to Pandora on a Transporter, I've decided I like the title + progress bar + time remaining on the left screen, and the spectrograph with channel name overlay on the right screen, because I thrive on information overload. We'll call this 'state 1'. In this state, if I hit the knob button it shifts the display around slightly. The left screen of course shows the "I like this song" menu, and the right one turns into the title overlaid with the spectrograph. That part is good, it makes sense and it looks nice. Let's call this 'state 2'. Now, from state 2 I select 'I don't like this song'. Pandora starts playing a new track and the left screen switches to the message 'Sorry, we won't play that song again'. This message is shown for a few seconds so the user can read it, which is good. HOWEVER during that few seconds, the right screen continues to show the information for the song that I just killed. After the few seconds is over and the Transporter returns to state 1, the right screen is updated to show the new song info. It seems like the new song info should appear in the appropriate place as soon as the new song begins playing.
The issue is the second screen doesn't update on setCurrentTitle. Any ideas Triode?
Since I was so verbose in the initial report, I'll clarify the last sentence of the second-to-last paragraph. It should read: After the few seconds is over, the Transporter returns to state 1 and the song display (now on the left screen) is correct.
Andy - is the: "Sorry, we won't play that song again" message displayed using showBriefly? If so this is really a consequence of bug 4009 - showBriefly on screen1 will block periodic updates to the second screen. I've been considering the best way round this, but haven't written any code yet as I wanted to think through the impact of the changes necessary. I'll get back to looking at this. If the error message was displayed via update [i.e via the normal lines function for the mode], then I suspect you would not see this.
Yes those messages use showBriefly.
Andy - do you use the block param to showBriely for them?
Yeah, the code looks like this: $client->showBriefly( { line1 => $line1, line2 => $line2, }, { block => 1, scroll => 1, callback => sub { # After displaying the rating confirmation message, we want to go back # to the Now Playing screen, not leave users sitting in the menu if ( blessed $client ) { Slim::Buttons::Common::pushMode( $client, 'playlist' ); } }, } );
You can't fool me with marking this as 'future'. It seems to be fixed as of right now...
Yep - if Andy has included the trunk changes for 4009 it should be fixed now!
Yep it's fixed.
This bug appears to have been fixed in the latest release! If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look. Make sure to include the version number of the software you are seeing the error with.