Bug 1676 - Force scroll on sb using remote control.
: Force scroll on sb using remote control.
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Player UI
: 6.0.3
: All All
: P5 enhancement (vote)
: Future
Assigned To: Blackketter Dean
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-06-15 01:32 UTC by Steven Moore
Modified: 2008-12-15 13:06 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Steven Moore 2005-06-15 01:32:11 UTC
Could a key combination be assigned to force a scroll of the squeezebox player display using the remote 
control?
Comment 1 Philip Meyer 2005-06-15 12:26:40 UTC
Is there a perl function that can be called to start the scroll effect?

This could then be mapped to any key combination that a user would like to use 
(such as holding down sleep).
Comment 2 KDF 2005-06-15 15:05:02 UTC
There are perl commands, but no button function associated with text control. 
For a mapping to be used, there would need to be a new function created in
Common.pm to trigger the text scrolling animation.  cc'ing triode on this for
input. Probably not hard to provide the hooks, but it may still require and edit
of the Default.map file for a user to activate it, since there really isn't room
for new button assignments.

Comment 3 Adrian Smith 2005-06-15 15:25:19 UTC
The new display code allows update() to be called with a "scrollMode" - this 
can be set for scrolling, scroll once (one time round and stop) and no 
scrolling.  It defaults to scrolling - ie to initiate scrolling if required.  
As all of the calls to update in the rest of slimserver don't set this the 
default is always used.

So an option would be to add a user customisable default for this which would 
allow the user to set it to something other than scrolling.  

Would scrollonce mode work for you Steven?  This would mean the text scrolls 
round once and stops rather than continuous scrolling.  If you want to read it 
again you would need to change screen back and forward, but it would mean that 
the steady state is always a stationary screen.  [advantage being no new 
button defintion required]

Dean/Kdf - you can try this out on my latest patch by changing the default of 
scrollMode in Slim::Player::Player::update to be 1 rather than 0.
Comment 4 KDF 2005-06-15 23:28:59 UTC
it seems that most modes supply a scrollmode, so the default seems to have no
practical effect.  I expect this could work, but a more global control would be
needed.  

I'm liking the ticker effect, however.
Comment 5 Steven Moore 2005-06-16 02:48:25 UTC
The scrollonce mode would work.
This would allow me to read the album name or sometimes the artist and the album name.
Once I have seen the name(s) there is no need to scroll again.
I like the ticker effect too, but in a darkened room I found it to be very distracting.
I set this to low priority so if it's going to take a lot of work to implement forget about it, there are more 
important tasks I'm sure. I just thought it might have been a simple call to a scroll routine.
I've set the interval to 1.5 minutes as most tracks are 3-4 minutes this will do.
It could be included in a later version perhaps.
Comment 6 Adrian Smith 2005-06-16 10:18:46 UTC
KDF - actually I overlooked the fact that any mode with updateInterval set 
will restart scrolling..!  This is because scrollonce mode exits scrolling 
when it completes and hence if update is called again it starts all over again.

I think it will be reasonably easy to change scrollOnce to stay in the scroll 
paused state and hence avoid another scroll, but still update the rest of the 
screen.  I'll look at in the next day or so and post another patch to the dev 
list...
Comment 7 Adrian Smith 2005-06-19 14:44:07 UTC
Please try latest patch posted to the dev list - this includes a per player 
setting for the "Scroll Mode" which can be set to "scroll once".  [and 
hopefull the scrolling internals now actually do this unless the text changes] 
Comment 8 KDF 2005-06-20 12:21:27 UTC
possible fix committed by triode at change 3463
Steven, please try the Jun 21 build of 6.1 and confirm if this is ok for you.
Comment 9 Adrian Smith 2005-06-20 12:27:19 UTC
Steven, please try setting the Scroll Mode (player - settings - display web 
page) to "Scroll once and stop".
Comment 10 Steven Moore 2005-06-21 08:49:07 UTC
Okay, DL 21/6/05 6.1 nightly.
This is working fine. I reset the pause between scrolls to 4 seconds. The display scrolls once at this time 
and does not scroll again until 4 seconds into the next track etc. Excellent.
I also noticed that if you are using the now playing screen saver and press the size button it causes a 
scroll 4 seconds later giving a force scroll function. Excellent again.
Comment 11 Adrian Smith 2005-06-26 06:02:28 UTC
I believe this can be closed now?
Comment 12 KDF 2005-06-26 10:43:59 UTC
I agree :)
Comment 13 Steven Moore 2005-06-26 11:19:48 UTC
Yes.
Comment 14 James Richardson 2008-12-15 13:06:44 UTC
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.