Review: Approve approve
Looks good. One remark: You've used "verseTag" as a stand-alone variable. I 
know it's in a Qt derived class, but it's a Python variable, I'd prefer it to 
be "verse_tag".
-- 
https://code.launchpad.net/~trb143/openlp/working/+merge/25849
Your team OpenLP Core is subscribed to branch lp:openlp.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to