-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110621/#review33070
-----------------------------------------------------------


Thanks for the patch :)


libs/mediacenter/playlistmodel.h
<http://git.reviewboard.kde.org/r/110621/#comment24453>

    better names would be firstIndex and secondIndex



libs/mediacenter/playlistmodel.cpp
<http://git.reviewboard.kde.org/r/110621/#comment24454>

    resetting the whole model is an expensive operation. Its better to use 
beginMoveRows for the source index.
    http://qt-project.org/doc/qt-4.8/qabstractitemmodel.html#beginMoveRows


- Shantanu Tushar


On May 23, 2013, 7:36 p.m., saurabh jain wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110621/
> -----------------------------------------------------------
> 
> (Updated May 23, 2013, 7:36 p.m.)
> 
> 
> Review request for Plasma, Shantanu Tushar and Sinny Kumari.
> 
> 
> Description
> -------
> 
> Reordering of the songs in the playlist is now possible using this patch. 
> Just drag and drop the item to the desired place (where you want to replace 
> it with the item at desired location). 
> 
> 
> Diffs
> -----
> 
>   libs/mediacenter/playlistmodel.h 2c1dc6b 
>   libs/mediacenter/playlistmodel.cpp 40b490c 
>   mediaelements/playlist/PlaylistDelegate.qml b1e6f40 
> 
> Diff: http://git.reviewboard.kde.org/r/110621/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> saurabh jain
> 
>

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

Reply via email to