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
Make most modules translatable #108
Conversation
07c37a2
to
58bf7f8
Compare
58bf7f8
to
7971c95
Compare
7971c95
to
626c0e6
Compare
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.
This looks great, thank you. I'll create the translation component on weblate and then translations can be submitted there.
Thanks for the merge @lassoan. I translated an easy one in French for a first experience, ArterialCalcificationPreProcessor namely. I have no idea what happens next, how and when it is included at runtime. Please inform briefly. I can continue later on without any hurry. Regards. |
You can now install the LanguagePacks extension, download translations, and select the application language. See https://github.com/Slicer/SlicerLanguagePacks/blob/main/HowToUse_fr.md |
Hi @lassoan, @pieper,
This PR is intended to make most modules of SlicerVMTK translatable. It is based on branch 'ParameterNodeWrapperAndOther' which is in PR #107, and not on the 'master' branch.
The modifications follow this discussion and the link to the translation guide.
I don't claim this PR is 100% fit for a merge. Please advise about required improvements.
@lassoan, please note I have felt free to modify 'ExtractCenterline' module also in 2c40a32. This patch can be easily dropped if you deem it necessary.
This PR is intended to be merged in 'Rebase and merge' mode. If PR #107 is not accepted for a merge, I can still modify this PR to rebase it on 'master'.
Your comments are welcome.
Regards.