-
Notifications
You must be signed in to change notification settings - Fork 744
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
GUACAMOLE-781: Add Czech translation #397
Conversation
GUACAMOLE-781 https://issues.apache.org/jira/browse/GUACAMOLE-781 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like a pretty complete translation - thank you, again! I do have a couple of requests for changes, here, before this can be merged:
- The biggest issue is the style - this needs to be consistent with the style of other code in Guacamole. There are two issues I see - first, indentations of lines should be four spaces (it looks like you did two). Second, the indentation of the second part of the translation (the actual translation) should be lined up and reasonably grouped together. For example:
"NAME" : "Name translation string"
"VERSION" : "Version translation string"
You can see full style guidlines, here: http://guacamole.apache.org/guac-style/
- I see a few instances where strings appear not to be translated. This may be because the translation is identical, in which case it should be kept, or because there is no translation, in which case it should be removed from this file.
I fixed idents and spaces, it should be ok now. |
Great, looks much better, thank you. A couple of other minor issues:
|
Sorry for that, my mistake again, well I cannot delete previous commits or edit it, so I just added last one named "Finall fix for cz.json GUACAMOLE-781" is that ok? |
Two things:
|
Czech translation of Guacamole client
Is this ok? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry @Vict0rC - One more thing that I didn't catch that needs to be tweaked, but looking very good!
…be only in base transaltion file.
Looks good to me, and @mike-jumper's utility agrees :-):
|
Hello,
I've just translated the guacamole interface into Czech. If anyone is interested...
https://github.com/Vict0rC/guacamole-client-cz
VictorC