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