Skip to content

i18n: Add Italian translation for localization#19

Open
GabriLesi wants to merge 1 commit into
1Computer1:mainfrom
GabriLesi:fix/it-IT
Open

i18n: Add Italian translation for localization#19
GabriLesi wants to merge 1 commit into
1Computer1:mainfrom
GabriLesi:fix/it-IT

Conversation

@GabriLesi

Copy link
Copy Markdown

Added Italian as a language to both the configuration and the fully translated JSON locale, as suggested in the open issue of this project.

I tested it locally, and just as a heads up: there is some minor string overflow on certain buttons like "Preferisci inserimento Han e Fu", simply because the text is longer in IT than in EN.

Added italian as a language to both the configuration and the fully translated JSON locale.
Tested locally
"characters": "Caratteri",
"circles": "Cerchi",
"closedOnly": "Solo mano chiusa",
"dealer": "Dealer",

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Should "Dealer" and "Non-Dealer" be translated?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

As far as i know my local game group (Tanuki Riichi) has always called them Dealer or Est even in italian language.
This is because the literal translation for "Dealer/Non Dealer" would be something like "Mazziere/Non Mazziere" but it's usually used in games with cards like Poker, hence why i prefered to keep this translation

},
"doubleriichi": {
"$": "Doppio Riichi",
"help": "Dichiarare riichi con una scommessa di 1000 punti quando si è in tenpai al proprio primo turno. Le chiamate la annullano. Non è più possibile modificare la mano; i pescapesca vengono scartati a meno che non portino alla vittoria."

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Typo here? "pescapesca"

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