Review Request 110604: Drag and Drop feature to remove items from playlist added.
Shantanu Tushar
shantanu at kde.org
Thu May 23 08:58:59 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110604/#review33013
-----------------------------------------------------------
mediaelements/playlist/Playlist.qml
<http://git.reviewboard.kde.org/r/110604/#comment24430>
Is this needed? Looks like a leftover
mediaelements/playlist/PlaylistDelegate.qml
<http://git.reviewboard.kde.org/r/110604/#comment24431>
extra whitespace, please remove.
mediaelements/playlist/PlaylistDelegate.qml
<http://git.reviewboard.kde.org/r/110604/#comment24432>
can you derive the value 200 from the playlist width? You can get it using listViewItem.ListView.view.width * 0.4
Experiment with values other than 0.4 to see what feels natural.
Also, remove the extra whitespace
mediaelements/playlist/PlaylistDelegate.qml
<http://git.reviewboard.kde.org/r/110604/#comment24433>
please use listViewItem.ListView.view.model.currentIndex instead of playlistModel. This helps in being agnostic about the model.
- Shantanu Tushar
On May 23, 2013, 8:16 a.m., saurabh jain wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110604/
> -----------------------------------------------------------
>
> (Updated May 23, 2013, 8:16 a.m.)
>
>
> Review request for Plasma, Shantanu Tushar and Sinny Kumari.
>
>
> Description
> -------
>
> Drag and Drop feature to remove items from playlist in PMC directly without interrupting the current session of media being played.
>
>
> Diffs
> -----
>
> libs/mediacenter/playlistmodel.h bedc26a
> mediaelements/playlist/Playlist.qml bbe97a7
> mediaelements/playlist/PlaylistDelegate.qml 7dac484
>
> Diff: http://git.reviewboard.kde.org/r/110604/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> saurabh jain
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20130523/03f99f79/attachment-0001.html>
More information about the Plasma-devel
mailing list