start music playback when using play button on playlist
ClosedPublic

Authored by astippich on Oct 14 2017, 2:07 PM.

Details

Summary

when no music is playing, klicking the play button from the playlist view does not start music playback.
fix this by passing down the playerControl to playlistentry and add the call to start playback

Diff Detail

Repository
R255 Elisa
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
astippich created this revision.Oct 14 2017, 2:07 PM
astippich edited the summary of this revision. (Show Details)Oct 14 2017, 2:09 PM
astippich added a reviewer: mgallien.
astippich changed the visibility from "Public (No Login Required)" to "Elisa (Project)".
mgallien changed the visibility from "Elisa (Project)" to "Public (No Login Required)".
mgallien added a project: Elisa.

Thanks a lot for this. This is a much needed modification. This adds consistency (the same button as the same effect).
While reading your patch, I thought that it might be better to use signals. This should help reduce tight coupling between the components.
Could you modify it to use signals to only have calls to manageAudioPlayer in MediaServer component ?

I believe I will mass modify all the other actions to follow the same pattern.

astippich updated this revision to Diff 20797.Oct 15 2017, 2:48 PM

Updated the patch to use signals instead of using manageAudioPlayer directly. The name of the id for PlayListEntry seems somewhat false, but I left it unchanged for now.

Since this is my first try at Qt/QML development and my very first contribution, you might want to take a closer look if it is really what you intended :)

mgallien accepted this revision.Oct 16 2017, 7:37 PM

Should I push the patch for you ?

This revision is now accepted and ready to land.Oct 16 2017, 7:37 PM
This revision was automatically updated to reflect the committed changes.