Review Request 116596: Fix for "Bug 329168 - It is impossible to remove a media from playlist without playing it first"

[Ashish Madeti]

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

(Updated March 5, 2014, 6:15 a.m.)


Review request for Plasma, Akshay Ratan, Shantanu Tushar, Sinny Kumari, and Sujith Haridasan.


Changes
-------

The previous patch didn't show the complete highlighted item (when it is cut off by the screen at the end).
Now it does.


Repository: plasma-mediacenter


Description
-------

Before this patch it was not possible to delete media (only using mouse) from a playlist without playing the media, as reported in Bug 329168 : https://bugs.kde.org/show_bug.cgi?id=329168

Now not only can you delete the media without playing it, it is easier to navigate the playlist then before.


Diffs (updated)
-----

  mediaelements/playlist/PlaylistDelegate.qml e1bd31d 
  mediaelements/playlist/Playlist.qml fd83c21 

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


Testing
-------

1.Deleted a song from a playlist without playing it.
2.Traversed the playlist without using keyboard or mouse scroll button.


Thanks,

Ashish Madeti

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140305/bb6c7642/attachment-0001.html>