Review Request 116596: Fix for "Bug 329168 - It is impossible to remove a media from playlist without playing it first"
Ashish Madeti
ashishmadeti at gmail.com
Thu Mar 6 14:40:58 UTC 2014
> On March 6, 2014, 2:23 p.m., Shantanu Tushar wrote:
> > I have merged this to master. Please note, please fill the bug field in reviewboard from the next time when you're fixing a bug.
I will definitely keep that in mind from next time. Thank you very much for helping me in fixing the bug.
- Ashish
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116596/#review52261
-----------------------------------------------------------
On March 6, 2014, 2:22 p.m., Ashish Madeti wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116596/
> -----------------------------------------------------------
>
> (Updated March 6, 2014, 2:22 p.m.)
>
>
> Review request for Plasma, Akshay Ratan, Shantanu Tushar, Sinny Kumari, and Sujith Haridasan.
>
>
> 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
> -----
>
> 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/20140306/016cc067/attachment.html>
More information about the Plasma-devel
mailing list