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

(Updated March 7, 2014, 3:44 p.m.)


Review request for Plasma, Shantanu Tushar and Sinny Kumari.


Changes
-------

Made changes suggested by Shantan.


Bugs: 331798
    http://bugs.kde.org/show_bug.cgi?id=331798


Repository: plasma-mediacenter


Description
-------

Patch for Bug 331798 : Playlist shows the wrong song as "now playing" (in red 
color) in some cases of reordering the playlist.


Diffs (updated)
-----

  libs/mediacenter/playlistmodel.h cb88c6c 
  libs/mediacenter/playlistmodel.cpp 6477047 
  mediaelements/playlist/Playlist.qml d578720 
  mediaelements/playlist/PlaylistDelegate.qml a5f6ebb 
  mediaelements/playlist/PlaylistDelegateLogic.js e69de29 

Diff: https://git.reviewboard.kde.org/r/116640/diff/


Testing
-------

1.Moved a song with index less than the currently playing media to some place 
below the currently playing media.
2.Moved a song with index greater than the currently playing media to some 
place above the currently playing media.
3.Moved the currently playing media to various places in the playlist.

In all the cases correct song is now showed as Now Playing i.e in "red" color.


Thanks,

Ashish Madeti

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to