Bug 11161 - Scrolling Text Timing
: Scrolling Text Timing
Status: CLOSED FIXED
Product: SqueezePlay
Classification: Unclassified
Component: UI
: unspecified
: PC Other
: P1 normal (vote)
: 7.5.0
Assigned To: Wadzinski Tom
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-02-23 12:41 UTC by ndijulio
Modified: 2010-04-08 17:24 UTC (History)
4 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description ndijulio 2009-02-23 12:41:52 UTC
The timing for the scrolling text (eg. now playing screen) needs to pause longer than 1sec.  I would propose having the text pause 3-4sec before scrolling left off screen.

Currently, the net effect is that the text is always scrolling.
Comment 1 Blackketter Dean 2009-02-23 14:29:41 UTC
In a related thread http://forums.slimdevices.com/showthread.php?t=60566 :

If you have a song name that is not terribly long, but long enough to scroll, along with an album title that is very long, it can take a very long time for the sing title to scroll back into view. I guess you would have the same problem if the song title was very long, but the album name was medium length.

and

The way I have seen other devices do this is to only scroll one line at a time. I wonder if this is appropriate in this case. This would make short lines always appear and if multiple lines are too long then each is scrolled in turn with the other lines remaining at the start.
Comment 2 ndijulio 2009-02-23 14:37:13 UTC
I think scrolling one item at a time would help tremendously.  I also think having the first portion whether it be the track, artist, or album have a longer rest period would help the screen feel less busy.
Comment 3 Weldon Matt 2009-04-14 10:25:22 UTC
Scrolling will be much less of an issue (at the very least) when the new design is implemented.  We'll use as much space as we can to prevent the need for the text to scroll (though of course there will always be occasionally very long strings)

See http://embargo.wiki.slimdevices.com/index.php/Now_Playing_design_spec
Comment 4 Richard Titmuss 2009-07-14 08:55:26 UTC
Tom, you did some work on this. You probably need to do a little more to get it right cross platform.
Comment 5 Richard Titmuss 2009-07-27 01:13:37 UTC
Reset priority before triage.
Comment 6 Wadzinski Tom 2009-08-05 04:22:45 UTC
 == Automatic comment from SVN commit #6915 to Repository: jive ==
https://svn.slimdevices.com/jive?view=revision&revision=6915

Bug:11161
Description:
- return to JIVE_FRAME_RATE / 2 frame rate
- Increased start and restart time.


 == Modified 1 files total ==
7.4/private-branches/fab4/squeezeplay/src/squeezeplay/src/ui/jive_label.c
Comment 7 Wadzinski Tom 2009-08-05 04:23:22 UTC
I put out a new version to try (r6915). It includes a ~2 second start (like
original Jive) then a ~3 second restart (new - original jive restart time was
less than a second). 

Also, I had recently (a few months ago) changed the scrolling to be smoother,
but on baby, at least, this causes the scrolling to unpleasantly stagger
occasionally while it's scrolling (I think due to occasionally long event loop
cycles). So, I returned to a scrolling framerate of JIVE_FRAME_RATE / 2, and a
higher "pixel shift per frame", like the original Jive behavior.

As Richard said, we'll need per device scrolling behavior also. This is not yet
in place.
Comment 8 Wadzinski Tom 2009-08-06 06:29:23 UTC
Moving this to P2 since it now seems acceptible for launch
Comment 9 Ben Klaas 2009-08-26 07:52:33 UTC
this is an administrative shuffle on priority fields to help make better judgment on the top end of the priority list. P4->P5, P3->P4, and P2->P3.
Comment 10 Wadzinski Tom 2009-10-03 12:48:12 UTC
I suggest we demo a few options.... Also, the scroll rate is affected by the device effective frame rate, which fluctuates quite a bit on the various devices, so we probably need. Also the scroll rate if done as pixels/time make the large text skin rate appear much slower than the small text skin.
Comment 11 Wadzinski Tom 2009-10-08 10:19:49 UTC
needs to be font size dependent
Comment 12 SVN Bot 2009-11-03 09:08:33 UTC
 == Auto-comment from SVN commit #8002 to the jive repo by tom ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8002 ==

Fixed Bug: 11161 +4
Description:
- scroll step now based on font size
Comment 13 Chris Owens 2010-04-08 17:24:36 UTC
This bug has been marked fixed in a released version of Squeezebox Server or the accompanying firmware or mysqueezebox.com release.

If you are still seeing this issue, please let us know!