Ended up being unsaved comment, however: "The automatic wrap puts all of text expect the last word to the same line, it does not look good at all. I believe manual newline is better in this case.
See the image for reference: https://drive.google.com/file/d/0B9y8rZiYItltamFQM0hOU0Nhd2s/view?usp=sharing " Diff comments: > > === modified file 'openlp/core/ui/shortcutlistform.py' > --- openlp/core/ui/shortcutlistform.py 2016-05-20 16:22:06 +0000 > +++ openlp/core/ui/shortcutlistform.py 2016-09-12 12:23:36 +0000 > @@ -426,11 +426,11 @@ > is_valid = False > if not is_valid: > text = translate('OpenLP.ShortcutListDialog', > - 'The shortcut "{key}" is already assigned to > another action, please' > - ' use a different shortcut.' > + 'The shortcut "{key}" is already assigned to > another action,\n' The automatic wrap puts all of text expect the last word to the same line, it does not look good at all. I believe manual newline is better in this case. See the image for reference: https://drive.google.com/file/d/0B9y8rZiYItltamFQM0hOU0Nhd2s/view?usp=sharing > + 'please use a different shortcut.' > > ).format(key=self.get_shortcut_string(key_sequence)) > > self.main_window.warning_message(translate('OpenLP.ShortcutListDialog', > 'Duplicate Shortcut'), > - text, for_display=True) > + text) > self.dialog_was_shown = True > return is_valid > -- https://code.launchpad.net/~suutari-olli/openlp/change-blank-to-desktop-hotkey-to-esc/+merge/305465 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
