[Sugar-devel] [PATCH sugar-toolkit] sugar.activity.i18n: add pgettext()
Simon Schampijer
simon at schampijer.de
Tue Feb 15 15:18:59 EST 2011
On 02/15/2011 01:37 PM, Simon Schampijer wrote:
> On 02/15/2011 01:22 PM, Sascha Silbe wrote:
>> Excerpts from Simon Schampijer's message of Tue Feb 15 19:00:34 +0100
>> 2011:
>>
>>> I think it makes sense to add a docstring that indicates what the method
>>> is doing and what the magic number is about, in the python3 patch I
>>> found:
>>
>> OK, will add it.
>>
>>> If I understand correctly in the support landed in Python3, so please
>>> add a FIXME that links to this ticket to indicate that.
>>
>> What ticket exactly? If you're talking about Py#2504 then I don't see
>> any indication that this is in Python 3 yet. The only thing we know for
>> sure is that Python 2.x will never have it.
>> But I can certainly add this information to a comment, not just the
>> patch description.
>
> Ok, I misread it then, but yes adding that info might be worthwhile so
> we don't forget about it.
>
>>> #: ../src/jarabe/frame/clipboardmenu.py:55
>>> #, fuzzy
>>
>> This confused me for quite a while, too, when I tested my patch. By
>> default fuzzy translations are NOT included in the compiled translations
>> file (.mo), so you need to remove the fuzzy flag (just remove the entire
>> line).
>>
>> Sascha
>
> Ahh, yeah - Gonzalo just pointed me to the same thing. Removing the
> fuzzy line makes it work. Will pootle add those fuzzy strings, too?
>
> Regards,
> Simon
An informal irc discussion came to the following conclusion:
a) include support for pgettext: do it if pootle does support it well
and is tested
b) which string to use to eject a device: make a poll (maybe using
'eject'/'expulsar' (used in GNOME) makes more sense in english, Sascha
pointed out that the use of 'eject' might have been avoided because
"current media (and esp. those used with XOs) don't actually eject",
maybe Gary knows more about it.
Regards,
Simon
More information about the Sugar-devel
mailing list