openlp-core team mailing list archive

[Tim Bentley <tim.bentley@xxxxxxxxx>]

Review: Needs Fixing

Looks good but all commits need tests to help build up our bank of tests,
These tests ideally should cover the area changed.
If you need help with the tests ask on our IRC channel.

I am wondering if we need a hover text as well as you change will impact Theme backgrounds  and Images.
-- 
https://code.launchpad.net/~tcf38012/openlp/stretchimage/+merge/323440
Your team OpenLP Core is subscribed to branch lp:openlp.