[Sugar-devel] [PATCH Sugar] sl#3316: Wireless modem error should refer to My Settings
Anish Mangal
anish at activitycentral.com
Mon Feb 6 07:38:55 EST 2012
Thx!
On Mon, Feb 6, 2012 at 10:34, Sascha Silbe <silbe at activitycentral.com> wrote:
> Excerpts from Anish Mangal's message of 2012-02-03 16:47:50 +0100:
>
>> The wireless modem error refers currently to Control Panel.
>
> There's more than one error alert that gets added by the GSM palette, so
> the above should read "One of the wireless modem error alerts", rather
> than "_The_ wireless modem error [alert]".
>
>
> [extensions/deviceicon/network.py]
>> @@ -752,8 +752,8 @@ class GsmDeviceView(TrayIcon):
>> error_handler=self.__connect_error_cb)
>> else:
>> self._palette.add_alert(_('No GSM connection available.'), \
>> - _('Create a connection in the ' \
>> - 'control panel.'))
>> + _('Create a connection in ' \
>> + 'My Settings.'))
>
> No need for explicit line continuation (the backslash-newline pair), as
> the open parenthesis implicitly continues the line. See also PEP-8,
> section "Code lay-out", "Maximum line length".
>
> Acked-by: Sascha Silbe <silbe at activitycentral.com>
>
> Sascha
>
> --
> http://sascha.silbe.org/
> http://www.infra-silbe.de/
More information about the Sugar-devel
mailing list