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 (it_it) translation #40

Merged
merged 3 commits into from
Jan 16, 2021
Merged

Add Italian (it_it) translation #40

merged 3 commits into from
Jan 16, 2021

Conversation

ZtereoHYPE
Copy link
Contributor

Adds italian translation to LambDynamicLights! :D
I translated all the strings as best as I could. Comparing it to where it should be in the menu, it all seems to make sense and be understandable, so I hope it's good for others too :)

Adds italian translation to LambDynamicLights :D
@ZtereoHYPE ZtereoHYPE changed the title Add Italian translation Add Italian (it_it) translation Nov 20, 2020
Copy link

@Riteo Riteo left a comment

Choose a reason for hiding this comment

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

Ci sono tante piccole finezze che ho segnalato. Non prenderle come rimproveri, ma più come consigli. In più, ci sono alcuni difetti nelle stringhe originali, alcuni dei quali ho risolto per evitare troppa confusione.
Grazie mille per aver contribuito alla traduzione di un'importantissima mod di Fabric!

src/main/resources/assets/lambdynlights/lang/it_it.json Outdated Show resolved Hide resolved
src/main/resources/assets/lambdynlights/lang/it_it.json Outdated Show resolved Hide resolved
src/main/resources/assets/lambdynlights/lang/it_it.json Outdated Show resolved Hide resolved
src/main/resources/assets/lambdynlights/lang/it_it.json Outdated Show resolved Hide resolved
src/main/resources/assets/lambdynlights/lang/it_it.json Outdated Show resolved Hide resolved
src/main/resources/assets/lambdynlights/lang/it_it.json Outdated Show resolved Hide resolved
src/main/resources/assets/lambdynlights/lang/it_it.json Outdated Show resolved Hide resolved
src/main/resources/assets/lambdynlights/lang/it_it.json Outdated Show resolved Hide resolved
src/main/resources/assets/lambdynlights/lang/it_it.json Outdated Show resolved Hide resolved
src/main/resources/assets/lambdynlights/lang/it_it.json Outdated Show resolved Hide resolved
Fixes many mistakes (oops) and improves translations a bunch thanks to @Riteo
@LambdAurora
Copy link
Owner

Quick note about the Canvas string: this one will become obselete soon-ish as Canvas is currently getting handheld dynamic lights, so LDL will switch to Canvas own methods if Canvas is present so the message will become irrelevant :)

@ZtereoHYPE
Copy link
Contributor Author

ZtereoHYPE commented Nov 22, 2020

Hold off the merge, there still a few issues we have to solve :D

Copy link

@Riteo Riteo left a comment

Choose a reason for hiding this comment

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

Dopo aver compilato la versione ho trovato veramente alcuni piccoli dettagli.
Dopo questo dovremmo aver finito!

src/main/resources/assets/lambdynlights/lang/it_it.json Outdated Show resolved Hide resolved
src/main/resources/assets/lambdynlights/lang/it_it.json Outdated Show resolved Hide resolved
src/main/resources/assets/lambdynlights/lang/it_it.json Outdated Show resolved Hide resolved
src/main/resources/assets/lambdynlights/lang/it_it.json Outdated Show resolved Hide resolved
@ZtereoHYPE
Copy link
Contributor Author

Everything should be ready now :D

@LambdAurora LambdAurora added to be merged translations Related to translations labels Nov 23, 2020
@LambdAurora LambdAurora merged commit 95df15f into LambdAurora:mc1.16 Jan 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
to be merged translations Related to translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants