Merge lp://qastaging/~tomasgroth/openlp/fix-theme-thumb into lp://qastaging/openlp

Proposed by Tomas Groth
Status: Superseded
Proposed branch: lp://qastaging/~tomasgroth/openlp/fix-theme-thumb
Merge into: lp://qastaging/openlp
Diff against target: 519 lines (+145/-98)
8 files modified
openlp/core/display/render.py (+70/-27)
openlp/core/ui/media/vlcplayer.py (+14/-13)
openlp/core/ui/themeform.py (+20/-15)
openlp/core/ui/thememanager.py (+12/-14)
openlp/core/ui/themewizard.py (+9/-10)
openlp/plugins/media/lib/mediaitem.py (+10/-9)
openlp/plugins/songs/lib/songstab.py (+1/-1)
tests/functional/openlp_core/ui/test_thememanager.py (+9/-9)
To merge this branch: bzr merge lp://qastaging/~tomasgroth/openlp/fix-theme-thumb
Reviewer Review Type Date Requested Status
Raoul Snyman Needs Fixing
Tim Bentley Approve
Review via email: mp+368571@code.qastaging.launchpad.net

This proposal has been superseded by a proposal from 2019-06-07.

Commit message

Add a webengine view for previewing themes.
Made VLC loading more robust.
A few minor fixes.

To post a comment you must log in.
Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

Linux tests failed, please see https://ci.openlp.io/job/MP-02-Linux_Tests/176/ for more details

Revision history for this message
Tim Bentley (trb143) :
review: Approve
Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

I like it, but you'll just need to fix the tests.

review: Needs Fixing
2879. By Tomas Groth

pep8

2880. By Tomas Groth

fix some broken tests

2881. By Tomas Groth

fix test

2882. By Tomas Groth

fix pep8

Unmerged revisions

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.