Bug 7289 - No feedback on Controller when the user hits their skip limit on Slacker.
: No feedback on Controller when the user hits their skip limit on Slacker.
Status: CLOSED FIXED
Product: MySqueezebox.com
Classification: Unclassified
Component: Slacker
: unspecified
: Macintosh Other
: -- normal (vote)
: 7.0
Assigned To: Andy Grundman
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-02-22 15:25 UTC by Blackketter Dean
Modified: 2008-05-15 13:04 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments
Patch (4.48 KB, text/plain)
2008-02-22 18:20 UTC, Andy Grundman
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Blackketter Dean 2008-02-22 15:25:12 UTC
Per Patrick Markel:  

When using the Duet I noticed that it does not message the user that they pegged their skipped limit.  It also got very sluggish and then I ended up getting captcha’d.  Notification of the skip limit appears fine on the Squeezebox.  

We need a pop-up toast that explains that they can't go on.
Comment 1 Andy Grundman 2008-02-22 18:20:06 UTC
Created attachment 2980 [details]
Patch

Here's a patch to update the Pandora, Slacker, and Rhapsody showBriefly's to work on Jive.  Ben, can you review these are the right format?

One issue is the showBriefly on Jive always stays for the same amount of time so longer text doesn't have time to scroll.
Comment 2 Blackketter Dean 2008-02-23 08:53:58 UTC
Ping, Ben: can you review the patch?
Comment 3 Ben Klaas 2008-02-23 16:47:54 UTC
patch works fine...

currently the amount of time a showBriefly is shown on screen is hard-coded to 3 seconds. I would vote for making the delay be configurable, but not until 7.0.1. Would require changes on both SC and Jive to work.
Comment 4 Andy Grundman 2008-02-23 17:09:01 UTC
OK.  Richard is going to look and see if it's easy to make the text wrap by passing a 'wrap' param or something.
Comment 5 Andy Grundman 2008-02-23 17:48:45 UTC
Fixed in change 17702.
Comment 6 Ben Klaas 2008-02-24 06:39:09 UTC
fwiw, text wrap in showBriefly's is bug 6907, targeted for 7.0.1, which on initial investigation I found to be non-trivial.
Comment 7 James Richardson 2008-05-15 13:04:17 UTC
This bug has recently been fixed in the latest release of SqueezeCenter 7.0.1

Please try that version, if you still see the error, then reopen this bug.

To download this version, please navigate to: http://www.slimdevices.com/su_downloads.html