-
Notifications
You must be signed in to change notification settings - Fork 111
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
Translations uptaded #12
Comments
I've added this commit pedro-mendonca/qtranslate-x@33e97c1 to optimize a string I've corrected before, where I've isolated the 'and' for translation. I think it looks and works better like this, cleaner code. |
Pedro, do you think comma is an international mark the same for all languages? If so, I would then suggest "%s and %s" instead of "%s, %s, %s, %s, %s and %s", just to be flexible on the number of functions listed, which may change later. It is not important, but would probably be even more clean? |
I really don't know. Thats why I thought it would be more clear and flexible for each translator to choose the best way to translate it in their language, in portuguese it's more or less like english in this case, %s, %s, %s, %, and %s, but there can be some cases that it fits better only with comas or something else. Of course the translator must understand clearly the statement in english to adapt it as he wish. |
Ok, I realised that if we ever need to change number of %s, we can adjust each po file as well without asking somebody to make a new translation, following whatever pattern will will see, so it is not a big deal. This way is definitely good for any language, while one %s might be indeed a problem for some - you are right. |
If the number of %s will probably change, of course you will need to update the language files, but that is a lot of unnecessary work. Instead of "Enable function names compatibility for %s, %s, %s, %s, %s and %s." |
May be just simply "Enable function name compatibility (%s)."? |
Yep, thats it. The list in %s of course must only use ; elements, no english, no text to translate. |
Hi Pedro, yes the number of functions already changed, I added qtrans_generateLanguageSelectCode, so our discussion here was not completely useless 👍 |
👍 |
Thursday Jan 29, 2015 at 09:46 GMT
Originally opened as qTranslate-Team/qtranslate-x#12
.pot catalog updated with new strings found.
Portuguese files updated also.
pedro-mendonca included the following code: https://github.com/qTranslate-Team/qtranslate-x/pull/12/commits
The text was updated successfully, but these errors were encountered: