----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118824/#review60644 -----------------------------------------------------------
This review has been submitted with commit f5118554d78a471c28afc1f6624909ba65ac6b31 by Shantanu Tushar on behalf of Ashish Madeti to branch vsrao-seriesbackend. - Commit Hook On June 19, 2014, 2:10 p.m., Ashish Madeti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118824/ > ----------------------------------------------------------- > > (Updated June 19, 2014, 2:10 p.m.) > > > Review request for Plasma, Shantanu Tushar and Sinny Kumari. > > > Bugs: 336414 > http://bugs.kde.org/show_bug.cgi?id=336414 > > > Repository: plasma-mediacenter > > > Description > ------- > > Whenever we added / removed any playlist, the highlight was stuck at one > point i.e changing the playlist after that was not changing the highlight. > > This was happening because we were directly assigning value to > "multiplePlaylistList.currentIndex" in qml code, which removed its previous > binding. Instead we should emit the currentIndexChanged() signal and let the > ListView do the rest. > > > Diffs > ----- > > libs/mediacenter/multipleplaylistmodel.cpp 87601f5 > mediaelements/playlist/MultiplePlaylists.qml 4b085fe > > Diff: https://git.reviewboard.kde.org/r/118824/diff/ > > > Testing > ------- > > Changed playlist after adding / removing a playlist. Now highlight is working > fine > > > Thanks, > > Ashish Madeti > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
