> On March 27, 2014, 1:05 p.m., Sebastian Kügler wrote: > > mediaelements/playlist/Playlist.qml, line 159 > > <https://git.reviewboard.kde.org/r/116874/diff/3/?file=257374#file257374line159> > > > > indenting is wrong now > > R.Harish Navnit wrote: > I have added a new diff file after I pulled the latest source and built > successfully. Please do check that > patch(https://git.reviewboard.kde.org/media/uploaded/files/2014/03/27/7f72b75b-6d72-4a5f-9460-699bb3846624__updated_fix.patch), > I've added {} to the if-statement in that one. It however, isn't disabled > when the playlist isn't active.
Can you update the diff in the review instead of attaching it? Otherwise its not possible to leave comments on code. - Shantanu ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116874/#review54293 ----------------------------------------------------------- On March 27, 2014, 11:22 a.m., R.Harish Navnit wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116874/ > ----------------------------------------------------------- > > (Updated March 27, 2014, 11:22 a.m.) > > > Review request for Plasma, Shantanu Tushar and Sinny Kumari. > > > Bugs: 330990 and Bug > http://bugs.kde.org/show_bug.cgi?id=330990 > http://bugs.kde.org/show_bug.cgi?id=Bug > > > Repository: plasma-mediacenter > > > Description > ------- > > Now the next and previous buttons do not work unless a media is playing. > > > Diffs > ----- > > mediaelements/playlist/Playlist.qml fd83c21 > > Diff: https://git.reviewboard.kde.org/r/116874/diff/ > > > Testing > ------- > > I've done the testing for this patch. > > These are the steps that I followed. > 1. Added a new playlist and added a few songs to the playlist. > 2. Re-opened Plasma Media Center > 3. Clicked the Next and Previous buttons. > 4. Nothing was played. > 5. Selected a song from the list and then tested the next and previous > buttons. > 6. The buttons worked seamlessly. > > > File Attachments > ---------------- > > pulled the latest source and made changes > > https://git.reviewboard.kde.org/media/uploaded/files/2014/03/27/7f72b75b-6d72-4a5f-9460-699bb3846624__updated_fix.patch > > > Thanks, > > R.Harish Navnit > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel