Review Request 116640: Fix for "Bug 331798 - Wrong media shown as current media in playlist after reordering"

Ashish Madeti ashishmadeti at gmail.com
Fri Mar 7 15:44:47 UTC 2014


-----------------------------------------------------------
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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140307/43596d40/attachment.html>


More information about the Plasma-devel mailing list