← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~trb143/openlp/bug-805084 into lp:openlp

 

Review: Approve
Nice fix!

Just a question, why did you break the docstrings up in two lines? Because I would leave them on one line.

In trunk you could loop through the service (when you are at the bottom and press the down arrow you jump to the first). Not sure if the others need it, I don't :-)
-- 
https://code.launchpad.net/~trb143/openlp/bug-805084/+merge/68938
Your team OpenLP Core is subscribed to branch lp:openlp.


References