Review: Needs Fixing
670 + if not bible:
671 + QtGui.QMessageBox.information(self.parent.mediaItem,
672 + translate('BiblesPlugin.BibleManager',
673 + 'No Bibles available'),
674 + translate('BiblesPlugin.BibleManager',
675 + 'There are no Bibles currently installed. Please use the '
676 + 'Import Wizard to install one or more Bibles.'))
677 + return None
Just a small one... instead of invoking this message box yourself, rather use
the new "openlp_information_message" signal, it'll anchor the message box to
the main window.
--
https://code.launchpad.net/~m2j/openlp/work/+merge/45287
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
