Review Request 116620: Bug 322820 - Playlist is not reachable from home screen

[Shantanu Tushar]

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

Ship it!


Ship It!

- Shantanu Tushar


On March 6, 2014, 4:17 p.m., Ashish Madeti wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116620/
> -----------------------------------------------------------
> 
> (Updated March 6, 2014, 4:17 p.m.)
> 
> 
> Review request for Plasma, Akshay Ratan, Shantanu Tushar, Sinny Kumari, and Sujith Haridasan.
> 
> 
> Repository: plasma-mediacenter
> 
> 
> Description
> -------
> 
> Playlist was not reachable from home screen as reported in Bug 322820 : https://bugs.kde.org/show_bug.cgi?id=322820
> 
> Default behavior of pmc when user clicks on empty area is to show the media player instance when some media is playing and if not then it does nothing.
> This patch will show the media controller when empty area is clicked and there is no media playing, so that user can now access playlists from there.Clicking again will hide the controller.
> 
> If we want to show the playlist page directly by clicking the empty area (i.e no media controller shown) then the 154th line can be changed to :
>   pmcPageStack.pushAndFocus(getPlaylist());
> 
> 
> Diffs
> -----
> 
>   shells/newshell/package/contents/ui/mediacenter.qml 49ea403 
> 
> Diff: https://git.reviewboard.kde.org/r/116620/diff/
> 
> 
> Testing
> -------
> 
> 1.On welcome screen, (when no media is playing) clicked on empty area and the media controller is showing and it goes into hiding by clicking again in empty area.
> 2.Left the media controller visible on welcome screen and started some media and then again went back to welcome screen, media controller is not visible now.
> 
> 
> Thanks,
> 
> Ashish Madeti
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140306/28eefaa2/attachment.html>