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

Improve Italian translations #153

Conversation

alessandrodalbello
Copy link
Contributor

These changes extend and improve the previous Italian translations. Most of the translations were missing, likely because copies were simply not present at that time. Some typos have been fixed and sentences adjusted to make them sound more natural.

Please, note that in the template recommended.html.eex, the two msgids "Grandmaster recommended" and "Chess Books" can not work in this order in Italian (and many other languages). To make the outcome meaningful, I had to swap the two translations (i.e. "Grandmaster recommended" translated as "Chess Books", and vice versa). This is because the same sentence is split into two chunks displayed in a given order, which simply doesn't work.
I strongly recommend introducing one single key ("Grandmaster recommended Chess Books") and adjusting the template to shrink the title in a way the text is distributed over two lines (if this is really the wanted effect). Needless to say, all translations should be adapted after that.


#, elixir-format
#: lib/listudy_web/templates/book/recommended.html.eex:1
msgid "Chess Books"
msgstr ""
msgstr "Suggeriti da Gran Maestri"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ArneVogel this and the following copy is where I had to swap the translations to make them meaningful, as explained in the PR description.

@ArneVogel ArneVogel merged commit 0318995 into ArneVogel:master May 21, 2023
@ArneVogel
Copy link
Owner

Thanks!

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

2 participants