Review Request 118824: Fix for "Bug 336414 - Playlist switching doesnt move highlight"
Commit Hook
null at kde.org
Sat Jun 21 09:29:40 UTC 2014
-----------------------------------------------------------
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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140621/d8b3301e/attachment.html>
More information about the Plasma-devel
mailing list