Review Request 114482: Plasma-mediacenter remember last played playlist

[Shantanu Tushar]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/114482/#review45725
-----------------------------------------------------------


Wait for few days and see if somebody has a better solution, otherwise we'll have to sin and use a Timer :(


mediaelements/playlist/MultiplePlaylists.qml
<http://git.reviewboard.kde.org/r/114482/#comment32660>

    Does someone have a better way of doing this without using a Timer?
    The issue here is that somehow if you set currentIndex to something, it resets back to 0, most probably because ListView does that after delegates are initialized.
    Using the Timer was the only way I could suggest Sujith right now :(


- Shantanu Tushar


On Dec. 15, 2013, 5:35 p.m., Sujith Haridasan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/114482/
> -----------------------------------------------------------
> 
> (Updated Dec. 15, 2013, 5:35 p.m.)
> 
> 
> Review request for Plasma, Shantanu Tushar and Sinny Kumari.
> 
> 
> Repository: plasma-mediacenter
> 
> 
> Description
> -------
> 
> With this change Plasma-mediacenter now remembers the last playlist used by user. Hence if there are playlists a, b, c and user selects playlist b and listens songs in it and closes Plasma-mediacenter. When Plasma-mediacenter relaunched, user can see the playlist b opened in the view.
> 
> 
> Diffs
> -----
> 
>   mediaelements/playlist/MultiplePlaylists.qml 42877f0 
> 
> Diff: http://git.reviewboard.kde.org/r/114482/diff/
> 
> 
> Testing
> -------
> 
> Tested with 3 playlists and it worked properly.
> 
> 
> Thanks,
> 
> Sujith Haridasan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20131215/887541fe/attachment.html>