Review Request 118406: Notify the user if the location containing the media is inaccessible.

[Shantanu Tushar]

> On June 5, 2014, 8:36 a.m., Thomas Pfeiffer wrote:
> > Usability review:
> > Since I lack the skills to picture it from the diff: When exactly is the notification shown? Is it shown as soon as the media is supposed to be played? If so, I think it could be done in a more subtle way: Grey out the entry in the playlist and just skip over it to the next entry int he playlist (like Amarok does) and show the message in a tooltip on mouseover on a pointer device or on tap on a touch device.
> > If it's already done in a similar way, then it's fine for me ;)
> 
> R.Harish  Navnit wrote:
>     I've added a screenshot which should help. This is a little different from the way amarok deals with the same error. If the media is inaccessible, the message is shown in the player screen. 
>     Is it okay, If I go ahead in the way I am or should look to implement something similar to what amarok does ??
> 
> Thomas Pfeiffer wrote:
>     Does PMC really have to be restarted to detect that the location is now accessible? That's not really a nice situation.
>     The whole message seems overly dramatic to me, especially in a playlist. Imagine you have a playlist with 50 songs in it, and three of them are on a location which isn't accessible. What would you prefer?
>     a) Having to close PMC, make the location accessible and then restart it
>     b) Just skip over those three songs and happily listen to the other 47
>     
>     If we're not in a playlist, the problem should rarely occur because the browser only shows currently accessible media anyway, right?
>
> 
> R.Harish  Navnit wrote:
>     So, like I'd mentioned in the mail(sorry about that). 
>     
>     1.PMC need not be restarted to get to detect that the location is accessible. I think the message might have been ambiguous. The user is expected to mount   
>       all drives containing media and then run the application again. 
>     2. You can skip over to the next media in a playlist. That's how I envisage the fix.
>     
>     Please let me know things have to be done differently.
> 
> Thomas Pfeiffer wrote:
>     The problem is that currently, the message is very scary. The skip controls are still active, but the user is presented with that very prominent error message telling her that she has to make sure that the file becomes accessible and restart the application. Plus, the playlist won't continue unless the user actively skips to the next entry, right?
>     The way Amarok does it is way less scary and disruptive since it silently skips. The only problem with Amarok's solution is that there is no way to find out why it skipped. That's why I suggested to offer a tooltip so users who wonder why a track was skipped can find out what the problem is. The message showing up in the tooltip could be like the one you've proposed, though I would change it to "Could not open the file [path]. To play this file, make sure it is accessible and restart Plasma Mediacenter"
>

Bah, this skipped me as well. There is absolutely no need to restart the app here, if the file is available, simply clicking play again should work.

Also, after Thomas pointed it out, in a playlist it actually makes sense to simply skip the missing media. However, if you play a single media (without the playlist) then this message seems appropriate.
@Thomas, does that sound right?

Finally, I'd suggest "and try again", insteasd of "and restart the application".


- Shantanu


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118406/#review59281
-----------------------------------------------------------


On June 5, 2014, 8:47 a.m., R.Harish  Navnit wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118406/
> -----------------------------------------------------------
> 
> (Updated June 5, 2014, 8:47 a.m.)
> 
> 
> Review request for Plasma, Shantanu Tushar and Sinny Kumari.
> 
> 
> Bugs: 333764
>     http://bugs.kde.org/show_bug.cgi?id=333764
> 
> 
> Repository: plasma-mediacenter
> 
> 
> Description
> -------
> 
> If a media(in a playlist) is located in an inaccessible location, then the user is notified about the same. 
> 
> 
> Diffs
> -----
> 
>   mediaelements/mediaplayer/MediaPlayer.qml 98f1d2c 
> 
> Diff: https://git.reviewboard.kde.org/r/118406/diff/
> 
> 
> Testing
> -------
> 
> 1. Load media to a playlist.
> 2. Unmount the device containing media.
> 3. Check if the user is notified of the location being inaccessible
>    --yes, the user is notified
> 4. Mount the device containing media and play a media from playlist.
>    -- The media plays properly.
> 
> 
> File Attachments
> ----------------
> 
> wihtout_i18n.png
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/06/05/3dc148a5-c4da-4d27-a713-e63922cbcef8__wihtout_i18n.png
> 
> 
> Thanks,
> 
> R.Harish  Navnit
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140605/8b6d9a2f/attachment.html>