Merge lp://staging/~ahayzen/qtubuntu-media/fix-1531296-random-mode-repeat-for-now into lp://staging/qtubuntu-media

Proposed by Andrew Hayzen
Status: Merged
Approved by: Jim Hodapp
Approved revision: 96
Merged at revision: 97
Proposed branch: lp://staging/~ahayzen/qtubuntu-media/fix-1531296-random-mode-repeat-for-now
Merge into: lp://staging/qtubuntu-media
Diff against target: 13 lines (+3/-0)
1 file modified
src/aal/aalmediaplaylistcontrol.cpp (+3/-0)
To merge this branch: bzr merge lp://staging/~ahayzen/qtubuntu-media/fix-1531296-random-mode-repeat-for-now
Reviewer Review Type Date Requested Status
Jim Hodapp (community) code Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+282323@code.staging.launchpad.net

This proposal supersedes a proposal from 2016-01-05.

Commit message

* Until bug 1518157 (RandomAndLoop playbackMode) is fixed set Random to be always looping due to bug 1531296

Description of the change

* Until bug 1518157 (RandomAndLoop playbackMode) is fixed set Random to be always looping due to bug 1531296

To post a comment you must log in.
Revision history for this message
Jim Hodapp (jhodapp) wrote : Posted in a previous version of this proposal

Looks good as a work around for making sure music-app doesn't have a regression from the way it works when it implemented playlists. This is a temporary fix until we decide how to solve the lack of this mode being possible with QtMultimedia's API.

review: Approve (code)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Jim Hodapp (jhodapp) wrote :

LGTM

review: Approve (code)
97. By Andrew Hayzen

* Merge of lp:qtubuntu-media

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches