Skip to content
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

Create pt.properties #20

Merged
merged 1 commit into from
Apr 11, 2013
Merged

Create pt.properties #20

merged 1 commit into from
Apr 11, 2013

Conversation

Stis
Copy link
Contributor

@Stis Stis commented Apr 11, 2013

The French translation.

The French translation.
Infocatcher added a commit that referenced this pull request Apr 11, 2013
@Infocatcher Infocatcher merged commit 8a883c7 into Infocatcher:master Apr 11, 2013
@Infocatcher
Copy link
Owner

Thanks!
But, I think, we should use "fr" instead of "fr-FR" as in Firefox itself, see
http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-mozilla-central-l10n/win32/xpi/
=> firefox-23.0a1.fr.langpack.xpi

@Infocatcher
Copy link
Owner

But, I think, we should use "fr" instead of "fr-FR" as in Firefox itself

Renamed: b62de34, 0995d69

@Stis
Copy link
Contributor Author

Stis commented Apr 11, 2013

Maybe you're right. Just an old habit cause of fr-BE (Belgium) or fr-CA (Canada).
Since i have you're wonderful add-on in french, do as you please. :)

@Infocatcher
Copy link
Owner

It's very easy to rename backward, if someone will translate to other French dialect. :)
And "fr" locale will be used for any "fr" or "fr-*" in general.useragent.locale preference.

@Stis
Copy link
Contributor Author

Stis commented Apr 11, 2013

Oh, not always the case. Thanks for the tip.

@Infocatcher
Copy link
Owner

I added new localizable strings and extension now can be localized using babelzilla:
http://www.babelzilla.org/forum/index.php?showtopic=7393

@Infocatcher
Copy link
Owner

I added new localizable strings, see #101 (comment) (also updated on babelzilla).
And now you can subscribe to above issue to receive notifications about future changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants