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 : [email protected]
Unsubscribe : https://launchpad.net/~openlp-core
More help : https://help.launchpad.net/ListHelp
