----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116656/#review52423 -----------------------------------------------------------
Instead of getting the index in QML using indexInSourceModelOf and then deleteing it in QML, I'd suggest you handle the deletion in the model itself. The rationale behind this is that QML is supposed to only contain UI logic. - Shantanu Tushar On March 8, 2014, 7:34 p.m., Ashish Madeti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116656/ > ----------------------------------------------------------- > > (Updated March 8, 2014, 7:34 p.m.) > > > Review request for Plasma, Shantanu Tushar, Sinny Kumari, and Sujith > Haridasan. > > > Bugs: 331820 > http://bugs.kde.org/show_bug.cgi?id=331820 > > > Repository: plasma-mediacenter > > > Description > ------- > > When you delete a media from playlist while searching, often some other media > gets deleted. > > > Diffs > ----- > > mediaelements/playlist/PlaylistDelegate.qml 644bfd0 > mediaelements/playlist/Playlist.qml dbca104 > libs/mediacenter/filterplaylistmodel.cpp e6f7053 > libs/mediacenter/filterplaylistmodel.h aa1284b > > Diff: https://git.reviewboard.kde.org/r/116656/diff/ > > > Testing > ------- > > Deleted some media, when filter text was not empty. Now, the correct media is > getting deleted from the playlist. > > > Thanks, > > Ashish Madeti > >
_______________________________________________ Plasma-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/plasma-devel
