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

Add Italian localization #130

Closed
wants to merge 6 commits into from
Closed

Add Italian localization #130

wants to merge 6 commits into from

Conversation

andreamazz
Copy link
Contributor

Hi. Since the project already supports localization I thought to start adding additional languages :)

This PR adds italian.

"Updated at %@" = "Aggiornato il %@";

"Checking Weather..." = "Controllo il Meteo...";
"Update Failed" = "Update non risucito";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should "risucito" be "riuscito"?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, sorry about that, typo. The sad thing is that I double checked the text twice 😓

@ldamon
Copy link

ldamon commented Jun 26, 2015

Looks good to me based on my beginner/intermediate college italian 😄 🇮🇹

@jakecraige
Copy link
Contributor

Thanks for this PR @andreamazz I'm looking over to today. Can you also localize https://github.com/thoughtbot/Tropos/blob/master/Resources/Other-Sources/en.lproj/InfoPlist.strings ?

"SlightSnow" = ", con bassa probabilità di nevicate";
"GoodSnow" = ", con alta probabilità di nevicate";

"Updated %@ at %@" = "Aggiornato il %@ alle %@";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rather than using the format string to do the localization what do you think about making the key for the localizations follow the same naming convention, Pascal case and add the missing english translations for them?

This would apply to a few in this file:

Updated %@ at %@ to UpdatedAtDateAndTime
Updated at %@ to UpdatedAtTime
Checking Weather... to CheckingWeather
Update Failed to UpdateFailed

@andreamazz
Copy link
Contributor Author

Happy to do it!
i've added the Info.plist strings, and changed the keys for the other translations. 👍

@@ -0,0 +1,2 @@
"CFBundleDisplayName" = "Tropos";
"NSLocationWhenInUseUsageDescription" = "Tropos userà la tua posizione solo per scoprire le condizioni meteo attuali e quelle previste nella la tua zona";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this have a period . on the end?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in 96184f8

@jakecraige
Copy link
Contributor

Thanks for your work on this. I've merged it in 5c062b6

@jakecraige jakecraige closed this Jun 26, 2015
@andreamazz
Copy link
Contributor Author

👍
Let me know if you need the translation for the AppStore meta as well.

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.

3 participants