openlp-core team mailing list archive

[Tim Bentley <tim.bentley@xxxxxxxxx>]

Review: Needs Fixing

It is difficult to test as I have a broken box.  
I have no webkit playback so I cannot test the code so text over video / alerts are a problem.

855 needs to be a to of class
1313 needs a comment as to what [1] means.  There are other places as well.
1508 Duplicate List
1844 Subset of Duplicate List
2177 Comment not needed

-- 
https://code.launchpad.net/~crichter/openlp/media_rewrite/+merge/82048
Your team OpenLP Core is subscribed to branch lp:openlp.