openlp-core team mailing list archive

[Azaziah <suutari.olli@xxxxxxxxx>]

See the comment for the right place.


"This used to be in self.main_window.warning_message, where it was causing traceback.

Does it still work?
I'm not exactly sure of what it's supposed to do to be honest.

Is there a reason why this could not be an a separate line?"

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-08-14 21:23:25 +0000
> @@ -430,7 +430,8 @@
>                               ' 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)
> +            for_display = True

This used to be in self.main_window.warning_message, where it was causing traceback.

Does it still work?
I'm not exactly sure of what it's supposed to do to be honest.

Is there a reason why this could not be an a separate line?

>              self.dialog_was_shown = True
>          return is_valid
>  


-- 
https://code.launchpad.net/~suutari-olli/openlp/change-blank-to-desktop-hotkey-to-esc/+merge/302896
Your team OpenLP Core is requested to review the proposed merge of lp:~suutari-olli/openlp/change-blank-to-desktop-hotkey-to-esc into lp:openlp.