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

Italian Translations #1200

Merged
merged 2 commits into from Nov 21, 2016
Merged

Italian Translations #1200

merged 2 commits into from Nov 21, 2016

Conversation

simonefranza
Copy link
Contributor

I translated the majority of the lines, but Manifest, Materials Excahange and Settings are still missing.

In Italian the substantives have a gender, male and female, and there are a lot of articles that we have to use depending on the gender. So there might be some problems with some messages. I downloaded the beta and saw that on the main screen in the English Version for Each Character you have for example:

  • Hunter
  • Awoken Female

But in Italian appears:

  • Cacciatrice (=Huntress)
  • Insonne (=Awoken)
    With this layout we have a problem with some messages for example:
    "Your loadout of 3 items have been transferred to your Awoken Female Hunter", beacuse in the Italian version appears "al tuo Insonne Cacciatrice" which is wrong for two reasons:
  1. The article doesn't match with the gender of the substantive
  2. The adjective (insonne) placement is wrong.

To fix this issue you can simply adopt the English style for the Italian on aswell.
This means: in the main screen would appear:

  • Cacciatore (Hunter)
  • Femmina Insonne (Awoken Female)

and on the message it would look like this:
"Il tuo loadout ... al tuo Cacciatore Insonne Femmina" which is right.

There's a similar issue with the Singular/Plural Variable from "Compare". In Italian the word Casco(Helmet) becomes Caschi in plural, so you cannot simply add an s at the end. Furthermore The plural articles depend from the gender so it's hard to use one single sentence for every case.

Applied: "Il tuo loadout di {{amount}} oggetti è stato trasferito al tuo {{store}}.",
AppliedItem: "Il tuo loadout di un oggetto è stato trasferito al tuo {{store}}.",
AppliedError: "Non è stato possibile trasferire nessuno degli oggetti del tuo loadout.",
AppliedWarn: "Il tuo loadout è stato parzialmente trasferito, ma per {{failed}} di {{total}} oggetti il trasferimento è fallito."},

Choose a reason for hiding this comment

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

A space is required before '}' object-curly-spacing

Debug: {
Dump: "Svuota informazioni nella console",
View: "Visualizza Item Debug Info"},
FarmingMode:{

Choose a reason for hiding this comment

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

Missing space before value for key 'FarmingMode' key-spacing

Help: "Aggiungere note a questa voce" },
Debug: {
Dump: "Svuota informazioni nella console",
View: "Visualizza Item Debug Info"},

Choose a reason for hiding this comment

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

A space is required before '}' object-curly-spacing

@bhollis
Copy link
Contributor

bhollis commented Nov 21, 2016

@simonefranza thanks for the insight! Just so you know, in a lot of places we're using the definitions from Bungie, though they offer certain variants based on gender. Sometimes we just need to choose the right variant. I'm not sure what facilities angular-translate gives us to vary our translations based on gender.

Compare: {
All: {
Singular: "Confronta tutti i {{type}} ({{quantity}})",
Plural: "Confronta tutti i {{type}} ({{quantity}})" },
Copy link
Contributor

Choose a reason for hiding this comment

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

@bhollis bhollis merged commit 464188a into DestinyItemManager:dev Nov 21, 2016
@delphiactual delphiactual mentioned this pull request Nov 21, 2016
6 tasks
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.

None yet

3 participants