[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