openlp-core team mailing list archive

[Azaziah <suutari.olli@xxxxxxxxx>]

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.