Review: Needs Fixing
Lines 494 onwards would be better formatted like below I think as it saves 
using \ and we're all used to having and reading translate() calls needing 
their own line.

self.tagTableWidget.horizontalHeaderItem(0).setText(
    translate('OpenLP.DisplayTagTab', 'Description'))

Correct/fix docstrings, don't just remove them (line 696) - could those lines 
just be moved into exec_()?

No comment on not code standardising the new dialog!
-- 
https://code.launchpad.net/~trb143/openlp/bugs/+merge/50528
Your team OpenLP Core is subscribed to branch lp:openlp.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to