Code review comment for lp:~tcf38012/openlp/stretchimage

Revision history for this message
Tim Bentley (trb143) wrote :

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.

review: Needs Fixing

« Back to merge proposal