New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Let add-ons get translated messages included in the core #4652
Comments
Comment 1 by norrumar on 2014-11-30 11:40 |
Hi: I agree, and work done by Alberto et. al. has paved the way for this function. |
CC @nvdaes, @ABuffEr
From: derekriemer [mailto:notifications@github.com]
Sent: Sunday, January 8, 2017 9:17 PM
To: nvaccess/nvda <nvda@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Subject: Re: [nvaccess/nvda] Let add-ons get translated messages included in the core (#4652)
Hi: I agree, and work done by Alberto et al has paved the way for this function.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub <#4652 (comment)> , or mute the thread <https://github.com/notifications/unsubscribe-auth/AHgLkJUI5oOVtFt-cwCnJ6TkgjXPDK1Gks5rQcLFgaJpZM4Ld8Qu> .
|
Anyone who wants a PR under their belt, this would be pretty easy to add. |
Hi, just to point out that perhaps it would be better to add something
similar to the translate function that I use in add-ons, instead of the
message function in which it's based, since message is for braille and
speaking, not for gui, and translate seems more generic:
def translate(text):
return _(text).
I think this function could be added in the addonHandler file.
Thanks.
El 09/01/2017 a las 6:28, derekriemer escribió:
… Anyone who wants a PR under their belt, this would be pretty easy to add.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#4652 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/APE65HSL1G57gHZNCmnzmjipGDsCBSJMks5rQcVvgaJpZM4Ld8Qu>.
|
Hi, I created an old branch on BitBuucket, so I can send a PR on GitHub now. |
You can see #6747 PR |
Closing as per #6747 (comment):
See that PR for further discussion. |
Thanks for your valuable comments.
I have created a public gist in case this is useful:
https://gist.github.com/nvdaes/00e6f53f36b2be7f794eb06629691531
El 06/02/2017 a las 7:58, James Teh escribió:
… Closed #4652 <#4652>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#4652 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/APE65G73KzkT687u9qO0e33R6s49C5lGks5rZsSDgaJpZM4Ld8Qu>.
|
Reported by norrumar on 2014-11-30 11:37
When creating add-ons, sometimes developers can use messages included in NVDA's core.
Alberto Buffolino, a member of the add-on team, creates a plugin to get the translations of these messages, which can be useful for translators, since they could skip duplicate localizations.
I suggest to add a getTRanslatedMessage function in add-on handler to get translated messages.
Thanks.
The text was updated successfully, but these errors were encountered: