-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116554/
-----------------------------------------------------------

(Updated March 16, 2014, 10:08 p.m.)


Review request for Plasma, Akshay Ratan, Shantanu Tushar, Sinny Kumari, and 
Sujith Haridasan.


Changes
-------

const (Something I need to remember always :P)
As for the dual getPlaylist() call, I substituted it with getMediaPlayer(), so 
the effect remains the same, while avoiding to load the playlist twice, and the 
media can be played through the command line (tested).


Repository: plasma-mediacenter


Description
-------

Referring to Bug #331040 The functionality of the media starting to play on the 
press of the "Play All" button has been implemented.
It has been noted and implemented that the media doesn't start over when new 
songs/videos are added to an existing playlist. 


Diffs (updated)
-----

  shells/newshell/package/contents/ui/mediacenter.qml bac33c2 
  browsingbackends/localfiles/localfilesabstractbackend.cpp faaafa7 

Diff: https://git.reviewboard.kde.org/r/116554/diff/


Testing
-------

The testing has been done through various test scenarios.


Thanks,

Harshit Agarwal

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

Reply via email to