Bug 319626 - Can't remove an another song from playlist without stopping the current media.
Summary: Can't remove an another song from playlist without stopping the current media.
Status: RESOLVED FIXED
Alias: None
Product: plasma-mediacenter
Classification: Plasma
Component: MediaPlayer (show other bugs)
Version: 1.0.0
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Shantanu Tushar
URL:
Keywords:
Depends on:
Blocks: 321239
  Show dependency treegraph
 
Reported: 2013-05-10 14:00 UTC by Saurabh Jain
Modified: 2013-06-16 18:40 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Remove another song from playlist without stopping the current media (4.95 KB, patch)
2013-05-15 20:25 UTC, Akshay Ratan
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Saurabh Jain 2013-05-10 14:00:58 UTC
In order to remove any song which is currently not being played, we first have to select that song (can be done only by clicking on that particular song). And on selection, the song gets played which will result in termination of the session of the song which we were listening to earlier. 

Reproducible: Always

Steps to Reproduce:
1. Play any song in the playlist.
2. To remove any other song, click on it.
3. On the selection of song (to be deleted), the session of the previous song is terminated.


Expected Results:  
Deletion of any song other than the one being played should not terminate the session of the present media.
Comment 1 Saurabh Jain 2013-05-10 14:29:01 UTC
I would like to present 2 solutions that can solve this problem. Here they are: 

1) As of now, selection of song and playing it is done simulateneously by just single clicking on it. Instead of doing this, we can fire an another event:- QEvent::MouseButtonDblClick and handle it at the places where we want to play the media. This will suffice to our purpose of deleting any song other than the one being played as now the selection of any song and playing it will be handled in a different manner using single mouse click and double mouse click respectively. 
 
- The same task can be accomplished using only keyboard navigation as well. Up/Down keys will select the songs and pressing the enter will play it. 

2) Next solution will result in adding an another button called the "play button" just before the "removeFromPlaylistButton". Now, any song in the playlist won't be played by just single clicking on it but by selecting or clicking on it first and then choosing one of the two buttons (as mentioned above), instead of. 

The one extra advantage of latter feature is that we can remove the multiple songs simultaneously. 

Both of the above mentioned features can also be used to reorder the songs in the playlist as both will handle the selection of songs in a different way and then after selecting it, just dragging it up or down in the playlist or firing the event- "QDragMoveEvent".
Comment 2 Shantanu Tushar 2013-05-12 17:09:25 UTC
We cannot use double clicks, because no one likes double tapping tablet screens. Instead what should happen is that the "remove" button should simply be shown for all songs. Thats the simplest solution I can think of right now.

Also, please do not discuss about reordering items in this bug, that should goto bug #319621
Comment 3 Akshay Ratan 2013-05-15 20:25:01 UTC
Created attachment 79907 [details]
Remove another song from playlist without stopping the current media

Now, in the playlist, "remove" sign is shown on every song , so the user has to just click that "sign" instead of clicking the song which earlier stopped the media.

So basically now the current media is not stopped when deleting an item from a playlist.
Comment 4 Akshay Ratan 2013-05-17 18:03:31 UTC
Commit ID :: db2ef72c14d70126c2ff2130a886d4538784753e 

Please consider closing it :)