← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~smpettit/openlp/ftw into lp:openlp

 

Review: Needs Fixing
Two things:

1) You also should add a status tip (this is the text in the lower left corner which is shown when you leave the cursor over a menu entry). I didn't notice this last time, sorry.
2) Line 84-86: the \ is not needed because the line you are breaking is encapsulated in brackets.

Rest looks good :-)
-- 
https://code.launchpad.net/~smpettit/openlp/ftw/+merge/69155
Your team OpenLP Core is subscribed to branch lp:openlp.


References