Bug 15822 - Improve streaming error messages for SP
: Improve streaming error messages for SP
Status: NEW
Product: SqueezePlay
Classification: Unclassified
Component: Diagnostics
: 7.5.x
: PC Debian Linux
: -- enhancement (vote)
: ---
Assigned To: Unassigned bug - please assign me!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-03-03 08:40 UTC by Alan Young
Modified: 2011-04-14 23:33 UTC (History)
2 users (show)

See Also:
Category: Feature


Attachments
patch for persistent popup message when duration is sent as -1 (482 bytes, patch)
2010-03-03 08:43 UTC, Ben Klaas
Details | Diff
Serevr-side patch (1.11 KB, patch)
2010-03-03 09:07 UTC, Alan Young
Details | Diff
Revised server-side patch (1.11 KB, patch)
2010-03-04 23:53 UTC, Alan Young
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alan Young 2010-03-03 08:40:55 UTC
Various error conditions can result in a text error message. For SP devices this is a so-called toast popup. These errors can be things like an un-playable file, connection failure for remote services and some other, more-obscure problem.

The error message currently includes the current title or URL. This is probably superfluous for SP, since this will usually be on the screen anyway.

The toast times out after 5 seconds. It might be appropriate for it to stay up until user-intervention or track-change, whichever comes first.
Comment 1 Ben Klaas 2010-03-03 08:43:17 UTC
Created attachment 6588 [details]
patch for persistent popup message when duration is sent as -1
Comment 2 Alan Young 2010-03-03 09:07:04 UTC
Created attachment 6589 [details]
Serevr-side patch
Comment 3 Alan Young 2010-03-04 23:53:54 UTC
Created attachment 6600 [details]
Revised server-side patch
Comment 4 Ben Klaas 2011-01-17 15:00:02 UTC
Alan, turns out the client-side portion of this patch is already checked into 7.6. Passing over to you.

After we have both sides of this checked in, let's do some testing with a -1 showBriefly and make sure it behaves as intended, and that the method of dismissing persistent showBrieflies makes sense from a UX perspective.
Comment 5 Alan Young 2011-01-19 03:14:02 UTC
Ben, there  a couple of problems:
1. There is no way to dismiss the popup (that I can find) on Fab4, either skin.
2. I think we need to define some conditions that could dismiss the toast automatically, such as a new track starting to play. Otherwise, a single bad track in a playlist will produce the toast and it will stay forever.
Comment 6 Alan Young 2011-01-19 03:52:46 UTC
In Jive, the Back button does not cancel the popup either, although it works on Baby.
Comment 7 Ben Klaas 2011-01-19 07:58:55 UTC
I'll look into adding window event listeners for dismissing the popup for touch and back button

Making the popup go away due to other events (e.g. track changing) seems much trickier on the surface, but I'll look into it.